Merged
Conversation
zuorenchen
added a commit
that referenced
this pull request
Apr 2, 2026
* Merge pull request #4 from zuorenchen/develop Pull develop branch with TVC, roll control, buoyancy force, and stochastic features * Update monte carlo sim with the refactored flight class * Add function in flight class to run a step of simulation for a time node at a time * End rocket step sim at impact event. Refactor _step_state initialization. Add post processing when _step_simulation finishes * Refactor step simulation to get sensor measurement after step * Update gnss sensor output to local relative frame and add velocity measurements * Fix stuff caught by copilot and run formatter * Rename pyproject name * Update rocketpy/simulation/flight.py to init plots by default Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Fix thrust3 after cut off Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Add commits to self.set_processed_results() that fails when results contain arrays * Update comment in gnss sensor measurement * Update stochastic_aero_surfaces.py docstring Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Add assert to check if max roll torque is positive * Update tvc class docstring * Update to assert coefficient_constants length * Add assert to check if gimbal range is positive * Remove min_time_step and max_time_step attributes from StochasticFlight class, inhert them from flight class * Fix volume handlings in rocket.py * Update docstring units * Add attribute to run simulation after flight class initialization by default for backward competability * Format * Fix: put volume parameter to the end with default * Refactor monte carlo class as simulation is run by default after flight class initialization --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.