Skip to content

485 wrap carmastate set functions#522

Merged
mattldawson merged 7 commits intomainfrom
485-wrap-carmastate_set-functions
Jul 30, 2025
Merged

485 wrap carmastate set functions#522
mattldawson merged 7 commits intomainfrom
485-wrap-carmastate_set-functions

Conversation

@K20shores
Copy link
Copy Markdown
Collaborator

@K20shores K20shores commented Jul 30, 2025 •
edited
Loading

  • adds a destructor for carma state
  • adds set_gas
  • adds set_detrain
  • adds set_bin

closes #485

@K20shores K20shores linked an issue Jul 30, 2025 that may be closed by this pull request
Wrap CARMASTATE_Set... functions #485
Closed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 30, 2025

📄 Documentation for this branch is available at: https://ncar.github.io/musica/branch/485-wrap-carmastate_set-functions/

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 30, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 45.94595% with 60 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.90%. Comparing base (2544c7d) to head (f1acb04).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
src/carma/interface.F90 38.46% 40 Missing ⚠️
src/carma/carma_state.cpp 54.76% 19 Missing ⚠️
src/carma/carma.cpp 50.00% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #522      +/-   ##
==========================================
+ Coverage   83.74%   83.90%   +0.15%     
==========================================
  Files          51       52       +1     
  Lines        5181     5319     +138     
==========================================
+ Hits         4339     4463     +124     
- Misses        842      856      +14

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@K20shores K20shores marked this pull request as ready for review July 30, 2025 20:52
@K20shores K20shores requested a review from mattldawson July 30, 2025 20:52
@mattldawson mattldawson merged commit aa48e53 into main Jul 30, 2025
71 checks passed
@mattldawson mattldawson deleted the 485-wrap-carmastate_set-functions branch July 30, 2025 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mattldawson mattldawson mattldawson approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Wrap CARMASTATE_Set... functions

3 participants

@K20shores @codecov-commenter @mattldawson