Skip to content

Simplify input for extensible interface reactions#1709

Merged
speth merged 2 commits intoCantera:mainfrom
speth:fix-1620
Jun 12, 2024
Merged

Simplify input for extensible interface reactions#1709
speth merged 2 commits intoCantera:mainfrom
speth:fix-1620

Conversation

@speth
Copy link
Copy Markdown
Member

@speth speth commented Jun 11, 2024

Changes proposed in this pull request

  • Add a test case for an extensible interface reaction
  • Fix the requirement of using an empty rate-constant field and needing the interface- prefix on the implementation of the rate parameterization for extensible reactions on interfaces.

If applicable, fill in the issue number this pull request is fixing

Closes #1620

Checklist

  • The pull request includes a clear description of this code change
  • Commit messages have short titles and reference relevant issues
  • Build passes (scons build & scons test) and unit tests address code coverage
  • Style & formatting of contributed code follows contributing guidelines
  • The pull request is ready for review

@speth speth added the Kinetics label Jun 11, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 11, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 75.72%. Comparing base (4565a55) to head (6fc7109).

Files Patch % Lines
src/kinetics/Reaction.cpp 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1709      +/-   ##
==========================================
- Coverage   75.72%   75.72%   -0.01%     
==========================================
  Files         443      443              
  Lines       60981    60982       +1     
  Branches     9557     9558       +1     
==========================================
  Hits        46178    46178              
  Misses      11777    11777              
- Partials     3026     3027       +1     

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

@speth speth marked this pull request as ready for review June 12, 2024 00:00
Copy link
Copy Markdown
Member

@ischoegl ischoegl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@speth speth merged commit 1b56a06 into Cantera:main Jun 12, 2024
@speth speth deleted the fix-1620 branch June 12, 2024 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Extensible Interface Reactions require empty parameters in YAML

2 participants