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

Allow users to limit the number of generations in a flux diagram #152

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

alongd
Copy link
Member

@alongd alongd commented May 16, 2024

Using the max_chemical_generations arg

@codecov-commenter
Copy link

codecov-commenter commented May 17, 2024

Codecov Report

Attention: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Project coverage is 47.36%. Comparing base (6176491) to head (c11d5ee).

Files Patch % Lines
t3/utils/flux.py 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #152      +/-   ##
==========================================
- Coverage   47.40%   47.36%   -0.05%     
==========================================
  Files          25       25              
  Lines        3390     3393       +3     
  Branches      912      914       +2     
==========================================
  Hits         1607     1607              
- Misses       1580     1583       +3     
  Partials      203      203              
Flag Coverage Δ
unittests 47.36% <0.00%> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@alongd alongd force-pushed the flux_generations branch 2 times, most recently from 5d1e68b to 5e2e38a Compare May 18, 2024 06:51
@alongd alongd force-pushed the flux_generations branch 2 times, most recently from 7436f36 to 5ad7c51 Compare July 9, 2024 17:50
@alongd alongd merged commit f3e8ebd into main Jul 10, 2024
5 checks passed
@alongd alongd deleted the flux_generations branch July 10, 2024 10:14
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.

2 participants