Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update ReactorNet drawing functions #1714

Merged
merged 3 commits into from
Jun 21, 2024
Merged

Conversation

speth
Copy link
Member

@speth speth commented Jun 18, 2024

Changes proposed in this pull request

  • Avoid maintaining redundant docstrings in drawnetwork.py -- just reference the docs for the member function versions
  • Fix ability to pass arguments to ReactorNet.draw to show composition for all reactors in the network
  • When drawing the state, put the reactor label inside the reactor boundary to avoid ugly overlaps

If applicable, provide an example illustrating new features this pull request is introducing
Graph generated by adding

mixer.draw(print_state=True, species="X")

to mix1.py:

rnet

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 force-pushed the update-draw-rnet branch from d85265f to 87218b5 Compare June 19, 2024 23:30
Copy link

codecov bot commented Jun 19, 2024

Codecov Report

Attention: Patch coverage is 87.50000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 72.82%. Comparing base (f4a88d2) to head (87218b5).

Files Patch % Lines
interfaces/cython/cantera/drawnetwork.py 80.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1714      +/-   ##
==========================================
- Coverage   72.83%   72.82%   -0.01%     
==========================================
  Files         379      379              
  Lines       53593    53595       +2     
  Branches     9122     9123       +1     
==========================================
+ Hits        39032    39033       +1     
- Misses      11609    11610       +1     
  Partials     2952     2952              

☔ 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 20, 2024 12:54
Copy link
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 @speth!

@ischoegl ischoegl merged commit e092189 into Cantera:main Jun 21, 2024
48 checks passed
@speth speth deleted the update-draw-rnet branch June 21, 2024 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants