Skip to content

Update duplicate reactions when serializing#1923

Merged
speth merged 1 commit intoCantera:mainfrom
speth:fix-duplicate-serialization
Jul 9, 2025
Merged

Update duplicate reactions when serializing#1923
speth merged 1 commit intoCantera:mainfrom
speth:fix-duplicate-serialization

Conversation

@speth
Copy link
Copy Markdown
Member

@speth speth commented Jul 8, 2025

Changes proposed in this pull request

  • When converting mechanisms to YAML, update duplicate reaction flags on all reactions to be self-consistent

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

Fixes #1922

If applicable, provide an example illustrating new features this pull request is introducing

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 Input Input parsing and conversion (for example, ck2yaml) Kinetics labels Jul 8, 2025
@speth speth force-pushed the fix-duplicate-serialization branch from 0bcd968 to 499076f Compare July 8, 2025 20:32
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 8, 2025

Codecov Report

Attention: Patch coverage is 76.92308% with 3 lines in your changes missing coverage. Please review.

Project coverage is 74.87%. Comparing base (0e55a49) to head (c06e8af).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/kinetics/Kinetics.cpp 75.00% 1 Missing and 2 partials ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1923   +/-   ##
=======================================
  Coverage   74.87%   74.87%           
=======================================
  Files         448      448           
  Lines       55734    55740    +6     
  Branches     9192     9195    +3     
=======================================
+ Hits        41730    41736    +6     
  Misses      10901    10901           
  Partials     3103     3103           

☔ 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.

@speth speth force-pushed the fix-duplicate-serialization branch from 499076f to c06e8af Compare July 8, 2025 22:07
@speth speth marked this pull request as ready for review July 9, 2025 00:05
Copy link
Copy Markdown
Member

@bryanwweber bryanwweber left a comment

Choose a reason for hiding this comment

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

Thanks Ray!

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.

Thanks, @speth ... this all looks good to me.

@speth speth merged commit 726dabc into Cantera:main Jul 9, 2025
94 of 95 checks passed
@speth speth deleted the fix-duplicate-serialization branch July 9, 2025 23:18
speth added a commit to speth/cantera that referenced this pull request Jul 11, 2025
Fixes some cases not handled correctly in Cantera#1923.
speth added a commit to speth/cantera that referenced this pull request Jul 11, 2025
Fixes some cases not handled correctly in Cantera#1923.
speth added a commit to speth/cantera that referenced this pull request Aug 24, 2025
Fixes some cases not handled correctly in Cantera#1923.
speth added a commit to speth/cantera that referenced this pull request Aug 25, 2025
Fixes some cases not handled correctly in Cantera#1923.
speth added a commit to speth/cantera that referenced this pull request Sep 23, 2025
Fixes some cases not handled correctly in Cantera#1923.
speth added a commit to speth/cantera that referenced this pull request Sep 23, 2025
Fixes some cases not handled correctly in Cantera#1923.
speth added a commit to speth/cantera that referenced this pull request Sep 23, 2025
Fixes some cases not handled correctly in Cantera#1923.
speth added a commit to speth/cantera that referenced this pull request Sep 24, 2025
Fixes some cases not handled correctly in Cantera#1923.
speth added a commit to speth/cantera that referenced this pull request Sep 29, 2025
Fixes some cases not handled correctly in Cantera#1923.
speth added a commit to speth/cantera that referenced this pull request Sep 30, 2025
Fixes some cases not handled correctly in Cantera#1923.
speth added a commit to speth/cantera that referenced this pull request Oct 1, 2025
Fixes some cases not handled correctly in Cantera#1923.
speth added a commit that referenced this pull request Oct 8, 2025
Fixes some cases not handled correctly in #1923.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Input Input parsing and conversion (for example, ck2yaml) Kinetics

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Generating reduced mechanisms causes "undeclared duplicate reaction" errors

3 participants