Skip to content

MNT: final fixes before next release#1044

Merged
Gui-FernandesBR merged 2 commits into
developfrom
mnt/final-fixes-before-next-release
Jul 4, 2026
Merged

MNT: final fixes before next release#1044
Gui-FernandesBR merged 2 commits into
developfrom
mnt/final-fixes-before-next-release

Conversation

@Gui-FernandesBR

Copy link
Copy Markdown
Member

Summary

Documentation, CHANGELOG.md and test polish ahead of the next release (v1.13.0), auditing every PR merged since v1.12.0. This "dá um tapa" on the changelog and docs and backfills missing tests. A separate PR will do the end-to-end bug hunt.

CHANGELOG reconciliation

Added to [Unreleased] (were missing):

Hygiene fixes: de-duplicated #958/#966; moved #974 to Fixed and #1041 to Removed only; removed the already-released #914 duplicate; pointed the logging (#973) and ND-interp (#969) entries at their PRs instead of the issues.

Backfilled the [v1.12.0] section with #940 (ThrustCurve timeouts), #941 (power_off/on_drag as Function + _input attrs) and #944 (radial-burn Jacobian) — these shipped in v1.12.0 code but were never logged.

Note on git topology: develop diverged from the release branch, so git tag --contains is misleading. Membership was verified by content (git show v1.12.0:<file>), which is how the already-released duplicates were identified.

Documentation

Tests

New regression tests (all passing): 3D ND-interp NaN outside convex hull (#969), abstract Parachute rejects instantiation (#958), Monte Carlo convergence stopping (#922), EnvironmentAnalysis surviving wind API (#1041), radial-burn grain geometry over time (#944), RingClusterMotor full flight (#924), discrete controller invoked exactly once per node (#949), acceleration-based parachute trigger deploys (#911). Plus backfill tests for ThrustCurve timeouts (#940) and drag Function/_input attributes (#941).

Verification

  • All new tests pass; the full modified test files (test_function.py, test_rocket.py, test_parachute.py, test_genericmotor.py, test_monte_carlo.py, test_ring_cluster_motor.py) pass with no regressions (--runslow included for the integration/env tests).
  • Sphinx is not installed locally, so the HTML docs build runs in CI; RST changes were reviewed for correctness and the new exceptions.rst uses automodule on an importable module. The HRRR snippet was intentionally made a static code-block (not jupyter-execute) to avoid a network-dependent build.

🤖 Generated with Claude Code

Documentation, CHANGELOG and test polish ahead of the next release
(v1.13.0), covering PRs merged since v1.12.0.

CHANGELOG
- Add missing entries: Individual Fins (#818), AIGFS/HRRR forecast
  models (#951), duplicate-controller fix (#949), Valkyrie flight
  example (#967).
- Hygiene: de-duplicate #958/#966, move #974 to Fixed and #1041 to
  Removed only, drop the already-released #914 duplicate, and point the
  logging (#973) and ND-interp (#969) entries at their PRs.
- Backfill #940/#941/#944 (shipped in v1.12.0 code but never logged)
  under the [v1.12.0] section.

Docs
- New exceptions reference page (rocketpy.exceptions) wired into the
  reference index; note UnstableRocketWarning in the rocket stability
  docs (#970).
- tanks.rst: switch examples to radius_function= and add a deprecation
  note (#957).
- forecast.rst: fix the HRRR example (missing code directive + stray
  sentence) (#951).
- airbrakes.rst: document discrete vs continuous controllers
  (sampling_rate=None) (#946).
- rocket_usage.rst: note that Parachute is now an abstract base;
  instantiate HemisphericalParachute (#958).

Tests
- New regression tests: 3D ND-interp NaN outside convex hull (#969),
  abstract Parachute cannot be instantiated (#958), Monte Carlo
  convergence stopping (#922), EnvironmentAnalysis surviving wind API
  (#1041), radial-burn grain geometry over time (#944), RingClusterMotor
  full flight (#924), discrete controller invoked once per node (#949),
  acceleration-based parachute trigger deploys (#911).
- Backfill regression tests for ThrustCurve API timeouts (#940) and
  power_off/on_drag Function objects + _input attributes (#941).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Gui-FernandesBR Gui-FernandesBR requested a review from a team as a code owner July 4, 2026 14:22
@codecov

codecov Bot commented Jul 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.52%. Comparing base (9cf3dd4) to head (4d6dad6).
⚠️ Report is 101 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1044      +/-   ##
===========================================
+ Coverage    80.27%   81.52%   +1.25%     
===========================================
  Files          104      118      +14     
  Lines        12769    15172    +2403     
===========================================
+ Hits         10250    12369    +2119     
- Misses        2519     2803     +284     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Gui-FernandesBR Gui-FernandesBR merged commit 51e862d into develop Jul 4, 2026
7 checks passed
@Gui-FernandesBR Gui-FernandesBR deleted the mnt/final-fixes-before-next-release branch July 4, 2026 14:46
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.

1 participant