ENH: Enable only radial burning

[caioessouza]

Pull request type

• Code changes (bugfix, features)

Checklist

• Lint (black rocketpy/ tests/) has passed locally
• All tests (pytest tests -m slow --runslow) have passed locally
• CHANGELOG.md has been updated (if relevant)

Current behavior

Rocketpy SolidMotor class simulates the grain regression either radially and axially by default.

This PR is related to the following issue:
ENH: Enable only radial burning #801

New behavior

Now it's possible to the user to change from the default behavior to a only radial regression by setting the new "only_radial_burning" optional parameter as True in the SolidMotor class.

Breaking change

• No

Additional information

Here is an example of the comparison between the default burning and the only radial one applied to the getting_started.ipynb simulation. The pytest tests haven't passed locally because of a windows problem.

[codecov]

Codecov Report

❌ Patch coverage is 45.83333% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.17%. Comparing base (9cf3dd4) to head (26580c4).
⚠️ Report is 1 commits behind head on develop.

Files with missing lines	Patch %	Lines
rocketpy/motors/solid_motor.py	45.83%	13 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #815      +/-   ##
===========================================
- Coverage    80.27%   80.17%   -0.10%     
===========================================
  Files          104      104              
  Lines        12769    12783      +14     
===========================================
- Hits         10250    10249       -1     
- Misses        2519     2534      +15     

☔ View full report in Codecov by Sentry.
📢 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.

[MateusStano]

Are we missing something in the HybridMotor class here? If we want to define a a HybridMotor with only radial burning garin how would we do that currently?

[Gui-FernandesBR]

Are we missing something in the HybridMotor class here? If we want to define a a HybridMotor with only radial burning garin how would we do that currently?

I recommend modifyng the hybrid class in another PR, it's easier to reivew it.

[Gui-FernandesBR]

Great PR.

I recommend keeping this PR to solidmotor (possibly renaming it).
Later you could extend the feature to also cover hybrid motors.

Also, I believe two other steps are important:

1. Update the Motor page on our documentation to include this new feature
2. Include new tests!!

[Gui-FernandesBR]

But if you really want to include hybrid motors here, not a problem

[Gui-FernandesBR]

Code looks better now, I think it is just a matter of taking a deeper look into the calculations and checking that everything is correct.

[Copilot]

Pull Request Overview

This PR enables users to restrict solid motor grain burning to only the radial direction by introducing a new only_radial_burn parameter. This enhancement allows for more accurate simulation of axially inhibited grains or hybrid motors where axial burning is not desired.

• Added only_radial_burn boolean parameter to both SolidMotor and HybridMotor classes with appropriate default values
• Modified geometry evaluation logic to conditionally disable axial burning based on the new parameter
• Implemented comprehensive test coverage for the new functionality

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
rocketpy/motors/solid_motor.py	Adds only_radial_burn parameter and implements conditional burning logic in geometry calculations
rocketpy/motors/hybrid_motor.py	Propagates only_radial_burn parameter to HybridMotor with default value of True
tests/unit/test_solidmotor.py	Adds unit tests for radial burn parameter effects and geometry evaluation
tests/unit/test_hybridmotor.py	Updates test time ranges and adds comprehensive tests for hybrid motor radial burn behavior
tests/integration/test_solidmotor.py	Adds new integration test file for SolidMotor info methods
CHANGELOG.md	Documents the new feature addition

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Gui-FernandesBR]

@phmbressan I will wait for your re-review before reviewing this one. I think you have more critical comments than me.

I would like to release this one soon.