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

DEV: Updates master for v1.6.2 #722

Merged
merged 8 commits into from
Nov 8, 2024
Merged

DEV: Updates master for v1.6.2 #722

merged 8 commits into from
Nov 8, 2024

Conversation

Gui-FernandesBR
Copy link
Member

No description provided.

Gui-FernandesBR and others added 7 commits October 19, 2024 16:35
* BUG: fix export ellipses to kml function

* BUG: fix export ellipses to kml function

* MNT: raise error if no apogee or impact data is found for plotting ellipses

* DEV: Adds #712 to changelog

* TST: increase monte carlo tests
* ENH: add structural to total mass ratio for motor and rocket

* ENH: adding structural mass ratio as an attribute of Motor

* ENH: add structural mass ratio to rocket

* ENH: capitalize words on prints

* DOC: adding structural mass ratio attribute to docstrings

* DOC: fix incorrect documentation names of attributes

* Remove erroneous comment

Co-authored-by: Gui-FernandesBR <[email protected]>

* DOC: modify changelog

* ENH: properly testing division by zero when computing the structural mass ratio

* MNT: make black

---------

Co-authored-by: Gui-FernandesBR <[email protected]>
Copy link

codecov bot commented Nov 7, 2024

Codecov Report

Attention: Patch coverage is 73.97260% with 19 lines in your changes missing coverage. Please review.

Project coverage is 76.03%. Comparing base (dde7ba4) to head (b838d53).
Report is 12 commits behind head on master.

Files with missing lines Patch % Lines
rocketpy/plots/monte_carlo_plots.py 52.63% 9 Missing ⚠️
rocketpy/simulation/monte_carlo.py 81.81% 6 Missing ⚠️
rocketpy/motors/motor.py 75.00% 2 Missing ⚠️
rocketpy/rocket/rocket.py 75.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #722      +/-   ##
==========================================
+ Coverage   75.93%   76.03%   +0.10%     
==========================================
  Files          99       94       -5     
  Lines       11222    10981     -241     
==========================================
- Hits         8521     8349     -172     
+ Misses       2701     2632      -69     

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

rocketpy/tools.py Show resolved Hide resolved
rocketpy/tools.py Show resolved Hide resolved
* DEV: updates workflows to include python 3.13

* DOC: edit doctest due to floating point precision.

---------

Co-authored-by: Pedro Bressan <[email protected]>
@Gui-FernandesBR Gui-FernandesBR merged commit f10f406 into master Nov 8, 2024
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Closed
Development

Successfully merging this pull request may close these issues.

3 participants