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 surface area models #2895

Closed
wants to merge 20 commits into from
Closed

Update surface area models #2895

wants to merge 20 commits into from

Conversation

jgostick
Copy link
Member

We need to do another quick release to update the surface area models to accept an amin argument. The default that was included in the last version was not general enough.

Copy link

codecov bot commented Jan 29, 2024

Codecov Report

Merging #2895 (1128d3a) into release (3dbe3e5) will decrease coverage by 0.2%.
Report is 55 commits behind head on release.
The diff coverage is 76.0%.

❗ Current head 1128d3a differs from pull request most recent head c6c2d5c. Consider uploading reports for the commit c6c2d5c to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##           release   #2895     +/-   ##
=========================================
- Coverage     89.9%   89.7%   -0.2%     
=========================================
  Files          148     148             
  Lines         8608    8660     +52     
=========================================
+ Hits          7742    7776     +34     
- Misses         866     884     +18     

@jgostick jgostick closed this Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants