-
-
Notifications
You must be signed in to change notification settings - Fork 248
Comparing changes
Open a pull request
base repository: RocketPy-Team/RocketPy
base: master
head repository: RocketPy-Team/RocketPy
compare: develop
- 15 commits
- 29 files changed
- 9 contributors
Commits on Mar 14, 2026
-
BUG: Fix hard-coded radius value for parachute added mass calculation (…
…#889) * Fix hard-coded radius value for parachute added mass calculation Calculate radius from cd_s using a typical hemispherical parachute drag coefficient (1.4) when radius is not explicitly provided. This fixes drift distance calculations for smaller parachutes like drogues. Formula: R = sqrt(cd_s / (Cd * π)) Closes #860 Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Address code review: improve docstrings and add explicit None defaults Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Add CHANGELOG entry for PR #889 Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Update rocket.add_parachute to use radius=None for consistency Changed the default radius from 1.5 to None in the add_parachute method to match the Parachute class behavior. This ensures consistent automatic radius calculation from cd_s across both APIs. Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Refactor Parachute class to remove hard-coded radius value and introduce drag_coefficient parameter for radius estimation Fix hard-coded radius value for parachute added mass calculation Calculate radius from cd_s using a typical hemispherical parachute drag coefficient (1.4) when radius is not explicitly provided. This fixes drift distance calculations for smaller parachutes like drogues. Formula: R = sqrt(cd_s / (Cd * π)) Closes #860 Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Add CHANGELOG entry for PR #889 Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Refactor Parachute class to remove hard-coded radius value and introduce drag_coefficient parameter for radius estimation MNT: Extract noise initialization to fix pylint too-many-statements in Parachute.__init__ Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> * Refactor environment method access in controller test for clarity * fix pylint * fix comments * avoid breaking change with drag_coefficient * reafactors Parachute.__init__ method * fix tests --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Co-authored-by: Gui-FernandesBR <guilherme_fernandes@usp.br>
Configuration menu - View commit details
-
Copy full SHA for 6ae5708 - Browse repository at this point
Copy the full SHA 6ae5708View commit details
Commits on Mar 20, 2026
-
ENH: Add guidelines for simulation safety, Sphinx documentation, and …
…pytest standards (GitHub Copilot) (#937)
Configuration menu - View commit details
-
Copy full SHA for 7d951f5 - Browse repository at this point
Copy the full SHA 7d951f5View commit details -
Configuration menu - View commit details
-
Copy full SHA for 22489fa - Browse repository at this point
Copy the full SHA 22489faView commit details -
ENH: Add explicit timeouts to ThrustCurve API requests and update cha…
…ngelog (#940) * Initial plan * ENH: Add explicit timeouts to ThrustCurve API requests Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> * DOC: Add timeout fix PR to changelog Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 8d49e5f - Browse repository at this point
Copy the full SHA 8d49e5fView commit details -
ENH: Restore power_off/on_drag as Function objects; add _input attrib…
…utes for raw user input and update changelog (#941) * Initial plan * ENH: Restore power_off/on_drag as Function, add _input attributes for raw user input Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> * DOC: Add PR #941 compatibility fix to changelog Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> * Update rocketpy/rocket/rocket.py Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> * MNT: ruff pylint --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Co-authored-by: MateusStano <mateusstano@usp.br>
Configuration menu - View commit details
-
Copy full SHA for 116f327 - Browse repository at this point
Copy the full SHA 116f327View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5f1a3eb - Browse repository at this point
Copy the full SHA 5f1a3ebView commit details -
Configuration menu - View commit details
-
Copy full SHA for 877e147 - Browse repository at this point
Copy the full SHA 877e147View commit details -
BUG: Fix incorrect Jacobian in
only_radial_burnbranch of `SolidMot……or.evaluate_geometry` (#944) * Initial plan * BUG: Fix incorrect Jacobian in only_radial_burn branch of evaluate_geometry Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 9d0ec38 - Browse repository at this point
Copy the full SHA 9d0ec38View commit details -
Configuration menu - View commit details
-
Copy full SHA for b205b75 - Browse repository at this point
Copy the full SHA b205b75View commit details -
Configuration menu - View commit details
-
Copy full SHA for dc6e878 - Browse repository at this point
Copy the full SHA dc6e878View commit details -
Configuration menu - View commit details
-
Copy full SHA for df52c15 - Browse repository at this point
Copy the full SHA df52c15View commit details
Commits on Mar 25, 2026
-
BUG: Add wraparound logic for wind direction in environment plots (#939)
* chore: added personal toolkit files * update branch name in workflow * chore: update toolkit files * Fix: add wraparound logic for wind direction and related tests * style: fix ruff formatting * Remove unused import Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * refactor: move repetitive logic into helper method * fix: update test logic in test_environment * add changelog entry --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 1f4f792 - Browse repository at this point
Copy the full SHA 1f4f792View commit details
Commits on Mar 27, 2026
-
Configuration menu - View commit details
-
Copy full SHA for d0ce62a - Browse repository at this point
Copy the full SHA d0ce62aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9cd2d34 - Browse repository at this point
Copy the full SHA 9cd2d34View commit details
Commits on Mar 29, 2026
-
ENH: Adaptive Monte Carlo via Convergence Criteria (#922)
* ENH: added a new function (simulate_convergence) * DOC: added a cell to show simulate_convergence function usage * TST: integration test for simulate_convergence * DOC: updated changelog for this PR * ENH: ran black to lint intg test file * new fixes thx to copilot comments * linted rocketpy/simulation/monte_carlo.py --------- Co-authored-by: Malmahrouqi3 <mohdsaid497566@gmail.com>
Configuration menu - View commit details
-
Copy full SHA for e0173e2 - Browse repository at this point
Copy the full SHA e0173e2View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff master...develop