DEV: Master to v1.9.0 #793
Merged
Conversation
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
* MNT: Update GitHub Actions workflows to use latest versions of actions * MNT: Update Codecov workflow to download latest coverage report and add overwrite option * DEV: remove flake8 from CI Ruff will take its place in a future PR
* DEV: adds ruff linter to the project IMPORTANT -> the ruff formatter is still not working properly * MNT: add ruff-report.txt to .gitignore * STY: ruff format * DEV: adds ruff format * STY: ruff format jupyter notebooks * DEV: update CI with ruff * DEV: update CI to use ruff for linting and formatting * DEV: remove unused enums for rocket coordinate system
* DOC: Add simulation file * DOC: Add simulation data * DOC : Add flight data * DOC: Update index * DOC : Run black * DEV: Update changelog * DOC: Delete unnecessary motor data, change to manufacturer data * DOC: Add simulation file * DOC: Add simulation data * DOC : Add flight data * DOC: Update index * DOC : Run black * DEV: Update changelog * DOC: Delete unnecessary motor data, change to manufacturer data * DOC: Ruff checks
* DEV: runs make format * DEV: update Makefile
* MNT: reimplement EmptyMotor class and refactor imports * MNT: add structural mass ratio property to EmptyMotor class * MNT: refactor EmptyMotor properties to use funcify_method and add tests * DEV: updates changelog
* BUG: update flight simulation logic to include burn start time in conditions * DEV: updates changelog
* DEV: add validation for aerodynamic surfaces in rocket plots * MNT: add conditional check for aerodynamic surfaces before drawing rocket * DEV: update CHANGELOG
* Squashed commit of the following: commit 4e0ef92 Merge: 0009c24 7864590 Author: Gui-FernandesBR <guilherme_fernandes@usp.br> Date: Mon Feb 10 05:05:23 2025 -0300 Merge branch 'develop' into enh/parallel_montecarlo commit 0009c24 Author: Lucas de Oliveira Prates <lucasdelprates@gmail.com> Date: Wed Dec 18 09:39:40 2024 -0300 BUG: fixing random number generator bug in StochasticRocket and issues inside methods of Components commit ebf6bd0 Merge: 00d9d02 2218f0f Author: Pedro Bressan <phmbressan@usp.br> Date: Mon Dec 16 22:35:18 2024 +0100 Merge remote-tracking branch 'origin/develop' into enh/parallel_montecarlo commit 00d9d02 Author: Pedro Bressan <phmbressan@usp.br> Date: Mon Dec 16 22:11:16 2024 +0100 MNT: Simplify Monte Carlo parallel export structure. commit 2218f0f Author: Yogiraj Gutte <53410698+yogirajgutte@users.noreply.github.com> Date: Mon Dec 16 08:22:33 2024 +0530 MNT: move piecewise functions to separate file (#746) * MNT: move piecewise functions to separate file closes #667 * improved import for linting * MNT: applying code formaters * ENH: simplifying and optimizing the function, implementing tests. * MNT: update changelog and apply changes suggested in review --------- Co-authored-by: Lucas Prates <57069366+Lucas-Prates@users.noreply.github.com> Co-authored-by: Lucas de Oliveira Prates <lucasdelprates@gmail.com> Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> commit 7a122ad Author: Lucas Gonçalves <lucasvgoncalves@tecnico.ulisboa.pt> Date: Mon Dec 16 02:49:34 2024 +0000 DOCS: Erebus11 - BME Suborbitals - 2022 flight simulation (#757) * DOCS: add data for bme suborbitals flight example * DOCS: add simulation file for bme suborbitals flight example * DOCS: Error in motor fixed * DOC: Improve flight examples documentation Update flight simulation documentation with improved markdown headers and replace matplotlib with Plotly for enhanced visualizations * DEV: update changelog * DOC: Update simulation and flight data * DOC: Add weather file for simulations * DOC: Updates to erebus flight sim * DOC: Update changelog * DOC: Fix title in camoes simul * DOC:Update flight data * DOC: Comparison plots * MNT: Delete unnecessary file * DOC: Update index * DOC: run Black * DEV: update changelog * DOC: small fix index --------- Co-authored-by: Gui-FernandesBR <guilherme_fernandes@usp.br> Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> commit ed6af68 Author: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Date: Sun Dec 15 23:43:31 2024 -0300 DOC: flight comparison improvements (#755) * DOC: Improve flight examples documentation Update flight simulation documentation with improved markdown headers and replace matplotlib with Plotly for enhanced visualizations * DEV: update changelog * DOC: Fix title in camoes simul * DOC: Update docs/requirements.txt to include new dependencies --------- Co-authored-by: LUCKIN13 <lucasvgoncalves@tecnico.ulisboa.pt> commit c7f1623 Author: Gui-FernandesBR <guilherme_fernandes@usp.br> Date: Sat Dec 14 23:55:09 2024 -0300 DEV: move CITATION file back to the root commit f407559 Author: Lucas Gonçalves <lucasvgoncalves@tecnico.ulisboa.pt> Date: Sun Dec 15 02:30:57 2024 +0000 DOC: Lince (Team STAR) 2023 flight sim (#752) * DOC: Add data for lince example * DOC: create simulation file * DOC: Update simulation file * DOC: Add euroc_2023 weather data * DOC: Add flight data * DOC: Update flight sim * DOC: Changelog update & Run black * DOC: Small fixes to fligh sim * DOC : Run black * DOC: Simulation fixes and 10000 index sim add * DOC: Update environments in flight documentation * DOC: fix Lince values in the rst file --------- Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Co-authored-by: Gui-FernandesBR <guilherme_fernandes@usp.br> commit 5d2fdef Author: Lucas Gonçalves <lucasvgoncalves@tecnico.ulisboa.pt> Date: Sun Dec 15 01:55:16 2024 +0000 DOCS : Andromeda 2022 flight simulation (#754) * DOC : Add weather file for simulation * DOC : Add data for simulation example * DOC: Add flight simulation example * DOC: Update index * DOC: Update Changelog * Update docs/examples/index.rst --------- Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> commit 17207ca Author: Caio Souza <99217921+caioessouza@users.noreply.github.com> Date: Sun Dec 8 03:15:00 2024 +0100 ENH: create a dataset of pre-registered motors. See #664 (#744) * ENH: Create a dataset of pre-registered motors. See #664 I followed the recommendation "Download and save several .eng files in the repo so we can install it along with the rocketpy package itself". The website thrustcurve.org was very useful to search for some .eng files. I mainly focused in some of the main brands on the market: Cesaroni, Aero Tech, Animal Motors and Loki. And also focused on classes K to M, because this is the main range of total impulse that I'm used to seeing in rocketry. I tried to pick motors with a difference of about 300~600Ns in total impulse. Some more improvements than can also be made following this issue are expanding the dataset for whole SolidMotor objects, more than only thrust curves. I think this would be what the recommendation "Save .json files with all the information we may find available on internet" could mean. I decided to go for the simple for now, but having the thrust curves is a good first step to implementing that in the future, which I would totally be able to do! * Update CHANGELOG.md * MNT: git rename motor eng files in data folder * ENH: Create a dataset of pre-registered motors. See #664 I followed the recommendation "Download and save several .eng files in the repo so we can install it along with the rocketpy package itself". The website thrustcurve.org was very useful to search for some .eng files. I mainly focused in some of the main brands on the market: Cesaroni, Aero Tech, Animal Motors and Loki. And also focused on classes K to M, because this is the main range of total impulse that I'm used to seeing in rocketry. I tried to pick motors with a difference of about 300~600Ns in total impulse. Some more improvements than can also be made following this issue are expanding the dataset for whole SolidMotor objects, more than only thrust curves. I think this would be what the recommendation "Save .json files with all the information we may find available on internet" could mean. I decided to go for the simple for now, but having the thrust curves is a good first step to implementing that in the future, which I would totally be able to do! Update CHANGELOG.md MNT: git rename motor eng files in data folder --------- Co-authored-by: Gui-FernandesBR <guilherme_fernandes@usp.br> commit f1b57ef Author: ArthurJWH <167456467+ArthurJWH@users.noreply.github.com> Date: Sat Dec 7 20:58:13 2024 -0500 DOC: add Defiance flight example (#742) * BLD: add a flight example to rocketpy "Defiance" rocket flight example was added to doc\examples as my (Arthur Hwang) challenge submission for the Team Recruitment * DOC: Add Defiance launch to flight examples graph. * Updates CHANGELOG --------- Co-authored-by: Pedro Bressan <phmbressan@usp.br> Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> DOC: add Defiance flight example (#742) * BLD: add a flight example to rocketpy "Defiance" rocket flight example was added to doc\examples as my (Arthur Hwang) challenge submission for the Team Recruitment * DOC: Add Defiance launch to flight examples graph. * Updates CHANGELOG --------- Co-authored-by: Pedro Bressan <phmbressan@usp.br> Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> commit fec6bf0 Author: Pedro Bressan <phmbressan@usp.br> Date: Sun Nov 24 18:53:58 2024 +0100 ENH: Allow for Alternative and Custom ODE Solvers. TST: Add slow testing for different ode solvers. MNT: Move ode solver validation to separate method. commit 8082710 Author: Pedro Bressan <phmbressan@usp.br> Date: Thu Sep 5 16:17:26 2024 -0300 MNT: fix pylint messages on file handling. commit df07955 Author: Pedro Bressan <phmbressan@usp.br> Date: Thu Sep 5 16:10:41 2024 -0300 MNT: add number of workers in parallel mode to prints. commit 1baedf6 Author: Pedro Bressan <phmbressan@usp.br> Date: Thu Sep 5 16:10:07 2024 -0300 MNT: simplify process start up syntax. commit 9f7325c Author: Pedro Bressan <phmbressan@usp.br> Date: Thu Sep 5 16:01:41 2024 -0300 MNT: avoid unnecessary reseedings on parallel monte carlo. commit 4246809 Author: Pedro Bressan <phmbressan@usp.br> Date: Thu Sep 5 16:00:03 2024 -0300 MNT: add index to outputs of monte carlo. commit e40a871 Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Aug 23 12:12:43 2024 -0300 DOC: improve docstrings regarding number of workers. commit 6fa90b7 Merge: d07fcc2 44beade Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Aug 23 12:04:10 2024 -0300 Merge remote-tracking branch 'origin/develop' into enh/parallel_montecarlo commit d07fcc2 Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Aug 23 12:04:00 2024 -0300 MNT: solve review comments on docstrings and code comments. commit 6dab002 Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Aug 23 11:57:25 2024 -0300 DOC: run 1000 MonteCarlo simulations for better documentatiion example. commit d22c957 Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Aug 23 10:17:59 2024 -0300 MNT: improve process ordering for spawned workers. commit 1e24643 Author: Pedro Bressan <phmbressan@usp.br> Date: Wed Aug 21 19:18:51 2024 -0300 FIX: return to multiprocess library for spawned process support. commit 0e4d243 Author: Pedro Bressan <phmbressan@usp.br> Date: Wed Aug 21 19:09:18 2024 -0300 MNT: solve number of processes issue on Windows. commit 5141791 Author: Pedro Bressan <phmbressan@usp.br> Date: Mon Aug 19 11:35:04 2024 -0300 MNT: improve object encoding and file handling. commit 8671e52 Author: Pedro Bressan <phmbressan@usp.br> Date: Sun Aug 18 19:46:43 2024 -0300 MNT: soft stop on parallel errors or interrupt. commit 6715791 Merge: b749979 8b4c14a Author: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Date: Sun Aug 18 09:44:03 2024 -0300 Merge branch 'develop' into enh/parallel_montecarlo commit b749979 Author: Pedro Henrique Marinho Bressan <87212571+phmbressan@users.noreply.github.com> Date: Fri Aug 16 22:59:06 2024 -0300 Update rocketpy/simulation/monte_carlo.py Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> commit 6061d3a Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Aug 16 22:37:41 2024 -0300 MNT: use standard multiprocessing with instance methods. Co-authored-by: MateusStano <mateusstano@usp.br> commit c3c6c3d Merge: cb88e69 3b61784 Author: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Date: Tue Aug 13 09:58:50 2024 -0300 Merge branch 'develop' into enh/parallel_montecarlo commit cb88e69 Merge: 3a08f48 4531ed5 Author: Pedro Bressan <phmbressan@usp.br> Date: Tue Aug 6 08:33:54 2024 -0300 Merge remote-tracking branch 'origin/develop' into enh/parallel_montecarlo commit 3a08f48 Merge: 049276d 0048737 Author: Pedro Henrique Marinho Bressan <87212571+phmbressan@users.noreply.github.com> Date: Mon Aug 5 17:45:55 2024 -0300 Merge pull request #649 from RocketPy-Team/mnt/parallel-refactor MNT: Refactor Parallel MonteCarlo and Stochastic Seeding commit 0048737 Author: Pedro Bressan <phmbressan@usp.br> Date: Sat Aug 3 22:39:38 2024 -0300 MNT: improve docstrings according to code reviews. commit fe7bad3 Author: Pedro Bressan <phmbressan@usp.br> Date: Sat Aug 3 22:27:26 2024 -0300 MNT: correct outdated docstrings and improve function naming. commit abe5747 Author: Pedro Bressan <phmbressan@usp.br> Date: Sat Aug 3 22:21:18 2024 -0300 FIX: stochastic model seed input not being used. commit d18408e Author: Pedro Bressan <phmbressan@usp.br> Date: Sat Aug 3 22:18:40 2024 -0300 MNT: improve random number generator naming. commit 26f692e Author: Pedro Bressan <phmbressan@usp.br> Date: Sat Aug 3 14:25:52 2024 -0300 MNT: fix printing and formatting issues. commit 004bf23 Author: Pedro Bressan <phmbressan@usp.br> Date: Sat Aug 3 13:16:28 2024 -0300 MNT: improve docstrings for parallel MonteCarlo. commit 8316993 Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Aug 2 23:13:25 2024 -0300 FIX: parallel random value generation not being independent. commit 049276d Author: Pedro Bressan <phmbressan@usp.br> Date: Mon Jul 29 11:11:18 2024 -0300 FIX: optional import handling of multiprocess module. commit 2169db1 Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Jul 26 18:11:13 2024 -0300 MNT: update optional dependencies for multiprocess. commit 2cdc95e Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Jul 26 17:59:57 2024 -0300 TST: fix testing for file Paths. commit d3a9004 Merge: 25a2fed a901b45 Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Jul 26 17:55:45 2024 -0300 Merge remote-tracking branch 'origin/develop' into enh/parallel_montecarlo commit 25a2fed Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Jul 26 17:54:46 2024 -0300 MNT: remove light mode and refactor I/O file handling. commit 8008aa7 Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Jul 26 11:09:14 2024 -0300 MNT: remove post processing scripts. commit 2a42b26 Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Jul 19 18:12:37 2024 -0300 FIX: small post merge corrections. commit c563472 Merge: 615a907 d977fbe Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Jul 19 17:28:41 2024 -0300 Merge remote-tracking branch 'origin/develop' into enh/parallel_montecarlo commit 615a907 Author: Pedro Bressan <phmbressan@usp.br> Date: Fri Jul 19 16:56:10 2024 -0300 MNT: run formatters and apply simple review suggestions. commit 3428608 Author: Bruno Sorban <brunosorban@usp.br> Date: Wed Jun 26 13:52:32 2024 +0200 Added time back to exported functions commit 01d77fa Author: Bruno Sorban <brunosorban@usp.br> Date: Wed Jun 26 10:07:49 2024 +0200 Encapsulated methods and reduced buffer size commit 2e56977 Author: Bruno Sorban <brunosorban@usp.br> Date: Wed Jun 19 17:27:01 2024 +0200 added input export to light mode commit cb276de Author: Bruno Sorban <brunosorban@usp.br> Date: Wed Jun 19 16:20:43 2024 +0200 Update sim counter for append mode commit 3114f81 Author: Bruno Sorban <brunosorban@usp.br> Date: Wed Jun 19 16:12:35 2024 +0200 Removed alpha serializer commit 4fe5314 Author: Bruno Sorban <brunosorban@usp.br> Date: Wed Jun 19 12:21:21 2024 +0200 Updated writer to write unpickled data commit d7ed4a1 Author: Bruno Sorban <brunosorban@usp.br> Date: Wed Jun 19 11:50:54 2024 +0200 Working shared memory with big buffer commit 1999c6d Author: Bruno Sorban <brunosorban@usp.br> Date: Tue Jun 18 17:39:59 2024 +0200 not deserializing data commit ceb1832 Author: Bruno Sorban <brunosorban@usp.br> Date: Wed Jun 12 21:41:33 2024 +0200 Added cpu limit commit d421a83 Author: Bruno Sorban <brunosorban@usp.br> Date: Wed Jun 12 18:58:33 2024 +0200 Working 2 way semaphore commit 2b8dc4b Author: Bruno Sorban <brunosorban@usp.br> Date: Tue Jun 11 14:45:32 2024 +0200 Updated start time commit 98ce6ba Author: Bruno Sorban <brunosorban@usp.br> Date: Tue Jun 11 13:37:38 2024 +0200 Centralized simulation control in SimCounter commit 38a29b1 Author: Bruno Sorban <brunosorban@usp.br> Date: Sun Jun 9 15:21:36 2024 +0200 Added documentation commit ee06b9d Author: Bruno Sorban <brunosorban@usp.br> Date: Sun Jun 9 15:04:11 2024 +0200 removed unsused file commit b3dcfc6 Author: Bruno Sorban <brunosorban@usp.br> Date: Sun Jun 9 15:01:54 2024 +0200 Updated append mode commit 918cbe0 Author: Bruno Sorban <brunosorban@usp.br> Date: Sun Jun 9 14:25:52 2024 +0200 Removed dev files commit 75bc96b Author: Bruno Sorban <brunosorban@usp.br> Date: Sun Jun 9 13:31:29 2024 +0200 removed test file commit 5a6547d Author: Bruno Sorban <brunosorban@usp.br> Date: Sun Jun 9 13:27:23 2024 +0200 Updated example notebook commit 1fe04e1 Author: Bruno Sorban <brunosorban@usp.br> Date: Sun Jun 9 13:27:12 2024 +0200 Added central post-processing script commit d57e436 Author: Bruno Sorban <brunosorban@usp.br> Date: Sun Jun 9 11:32:37 2024 +0200 Enabled number of workers control commit 9cef636 Author: Bruno Sorban <brunosorban@usp.br> Date: Sun Jun 9 10:26:06 2024 +0200 Added append logic to h5 file commit 175a025 Author: Bruno Sorban <brunosorban@usp.br> Date: Wed Jun 5 18:19:21 2024 +0200 one lock per file commit 1146e20 Author: Bruno Sorban <brunosorban@usp.br> Date: Wed Jun 5 18:09:06 2024 +0200 using queue to manage simulations commit be32a75 Author: Bruno Sorban <brunosorban@usp.br> Date: Tue May 21 21:15:33 2024 +0200 Added post-processing scripts commit 6ea6ef8 Author: Bruno Sorban <brunosorban@usp.br> Date: Tue May 21 17:18:58 2024 +0200 Style changes commit 46f5f00 Author: Bruno Sorban <brunosorban@usp.br> Date: Tue May 21 17:09:14 2024 +0200 Enable both export modes for serial and parallel commit 1b50e94 Author: Bruno Sorban <brunosorban@usp.br> Date: Thu May 9 16:36:50 2024 +0200 Write mode added commit 2927448 Author: Bruno Sorban <brunosorban@usp.br> Date: Thu May 9 12:17:22 2024 +0200 Working version with shared objects commit 6fbe0f7 Author: Bruno Sorban <brunosorban@usp.br> Date: Thu May 9 11:17:46 2024 +0200 added counter commit 2d5ff8d Author: Bruno Sorban <brunosorban@usp.br> Date: Sat May 4 14:17:38 2024 +0200 Basic paralllel structure added * MNT: fix merge errors * DEV: updates changelog * MNT: implementing back the last version of MonteCarlo class prior to conflict solving * MNT: linters * DOC: mention parallel in docs * MNT: ruff --------- Co-authored-by: Lucas de Oliveira Prates <lucasdelprates@gmail.com> Co-authored-by: MateusStano <mateusstano@usp.br>
* BUG: fixed AGL altitude in _FlightPrints.events_registered * updeted CHANGELOG
…cket object without rail buttons (#786) * DOC: fixed a typo in funcify_method() description * TST: created test for get_instante_attributes() with flight without rail buttons * BUG: fixed __calculate_rail_button_forces() by assigning a Function(0) to null_force instead of an empty array * DEV: updates CHANGELOG
* DEV: adds 'pip install -r requirements-tests.txt' recipe to 'make install' target on Makefile Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com>
…uniform distributions in Stochastic Classes (#783) * wind factor bug corrected the wind factor wasn't applied to the env.wind_velocity properties * BUG: StochasticModel visualize attributes of a uniform distribution It showed the nominal and the standard deviation values and it doesn't make sense in a uniform distribution. In a np.random.uniform the 'nominal value' is the lower bound of the distribution, and the 'standard deviation' value is the upper bound. Now, a new condition has been added for the uniform distributions where the mean and semi range are calculated and showed. This way the visualize_attribute function will show the whole range where the random values are uniformly taken in * variable names corrections * Corrections requested by the pylint test * ENH: more intuitive uniform distribution display in StochasticModel Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> --------- Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #793 +/- ##
==========================================
- Coverage 80.06% 79.09% -0.97%
==========================================
Files 95 96 +1
Lines 11350 11583 +233
==========================================
+ Hits 9087 9162 +75
- Misses 2263 2421 +158 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
* wind factor bug corrected the wind factor wasn't applied to the env.wind_velocity properties * BUG: StochasticModel visualize attributes of a uniform distribution It showed the nominal and the standard deviation values and it doesn't make sense in a uniform distribution. In a np.random.uniform the 'nominal value' is the lower bound of the distribution, and the 'standard deviation' value is the upper bound. Now, a new condition has been added for the uniform distributions where the mean and semi range are calculated and showed. This way the visualize_attribute function will show the whole range where the random values are uniformly taken in * variable names corrections * Corrections requested by the pylint test * ENH: more intuitive uniform distribution display in StochasticModel Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> * ENH: Eccentricities added to the StochasticRocket class A bug has been corrected in Flight class and an enhancement has been performed in the Rocket class as well * BUG: thrust eccentricity bug corrected eccentricity_y was defined by x coordinate and eccentricity_x was defined by y coordinate * BUG: Undo some Rocket class changes * ENH: add eccentricities to StochasticRocket * BUG: fix MonteCarlo eccentricity inputs * ENH: pylint and ruff recommended changes * TST: fix tests with eccentricity --------- Co-authored-by: Gui-FernandesBR <guilherme_fernandes@usp.br>
Gui-FernandesBR
previously approved these changes
Mar 23, 2025
* wind factor bug corrected the wind factor wasn't applied to the env.wind_velocity properties * BUG: StochasticModel visualize attributes of a uniform distribution It showed the nominal and the standard deviation values and it doesn't make sense in a uniform distribution. In a np.random.uniform the 'nominal value' is the lower bound of the distribution, and the 'standard deviation' value is the upper bound. Now, a new condition has been added for the uniform distributions where the mean and semi range are calculated and showed. This way the visualize_attribute function will show the whole range where the random values are uniformly taken in * variable names corrections * Corrections requested by the pylint test * ENH: add multiplication for 2D functions in rocketpy.function Added the ability to multiply functions with 2D domains in the __mul__ function * ENH: StochasticAirBrakes class created The StochasticAirBrakes class has been created. The __init__.py files in the stochastic and rocketpy folders have also been modified accordingly to incorporate this new class * ENH: set_air_brakes function created This functions appends an airbrake and controller objects previuosly created to the rocket * ENH: add StochasticAirBrake to rocketpy.stochastic_rocket Some functions has been modified and other has been created in order to include the new StochasticAirBrakes feature into the StochasticRocket class. A new function named 'add_air_brakes' has been created to append a StochasticAirBrakes and Controller objects to the StochasticRocket object. A new function '_create_air_brake' has been introduced to create a sample of an AirBrake object through a StochasticAirBrake object. Enventually, the 'create_object' function has been modified to add the sampled AirBrakes to the sampled Rocket * BUG: StochasticAirBrake object input in _Controller When defining the _Controller object a StochasticAirBrake was input. This is already corrected and a AirBrake object is now introduced * ENH: add time_overshoot option to rocketpy.stochastic_flight Since the new 6D47 StochasticAirBrake class is defined, we need the 'time_overshoot' option in the Flight class to ensure that the time step defined in the simulation is the controller sampling rate. The MonteCarlo class has had to be modified as well to include this option. * DOC: StochasticAirBrakes related documentation added Documentation related to the StochasticAirBrakes implementation has been added in StochasticAirBrakes, StochasticRocket and Rocket classes. * ENH: pylint recommendations done * ENH: Reformatted files to pass Ruff linting checks * ENH: Update rocketpy/stochastic/stochastic_rocket.py Unnecessary comment Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> * DOC: improve drag curve factor definition in StochasticAirBrakes * ENH: Change assert statement to if Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> * DOC: better explanation of __mul__ function Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> * ENH: delete set_air_brakes function for simplicity * DOC: CHANGELOG file updated --------- Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com>
MateusStano
added a commit
that referenced
this pull request
Apr 21, 2025
commit c674725 Author: Kevin Alcañiz <kevinalcaniz22@gmail.com> Date: Sat Apr 12 13:40:25 2025 +0200 ENH: Introduce Net Thrust with pressure corrections (#789) * wind factor bug corrected the wind factor wasn't applied to the env.wind_velocity properties * BUG: StochasticModel visualize attributes of a uniform distribution It showed the nominal and the standard deviation values and it doesn't make sense in a uniform distribution. In a np.random.uniform the 'nominal value' is the lower bound of the distribution, and the 'standard deviation' value is the upper bound. Now, a new condition has been added for the uniform distributions where the mean and semi range are calculated and showed. This way the visualize_attribute function will show the whole range where the random values are uniformly taken in * variable names corrections * Corrections requested by the pylint test * ENH: Add pressure corrections for thrust in SolidMotor The thrust generated by a SolidMotor is now adjusted for the atmospheric pressure. To achieve that, a new attribute, 'vacuum_thrust', has been created. The 'net_thrust' is the result of 'vacuum_thrust' minus the atmospheric pressure multiplied by the nozzle area. * ENH: pylint recommendations done * ENH: net thrust method extended to the rest of the motor classes * BUG: __post_processed_variables inconsistent array * ENH: ruff reformatting * Update rocketpy/motors/motor.py Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> * ENH: Avoid breaking change * ENH: Pressure Thrust method added * BUG: call to the thrust function wrong * BUG: pressure thrust evaluated when motor is turned off * ENH: CHANGELOG updated * DOC: definition of exhaust velocity improved --------- Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> commit 9f2644a Author: Lucas Prates <57069366+Lucas-Prates@users.noreply.github.com> Date: Sat Apr 12 11:27:53 2025 +0200 ENH: Implement Multivariate Rejection Sampling (MRS) (#738) * ENH: implementing a draft version of the Multivarite Rejectio Sampler (MRS). * MNT: quick notebook to test MRS during development * MNT: refactoring class to match review suggestions * ENH: add comparison prints, plots and ellipses to MonteCarlo and finally checks in MRS * MNT: add MultivariateRejectionSampler class to inits and apply format * DOC: writting .rst documentation for MRS * MNT: adding pylint flags to skip checks * DOC: completing missing sections in mrs.rst * DOC: add changelog and apply sugestions in MRS class * DOC: apply suggestions to the MRS.rst * MNT: use Union instead of | for type hinting since we have to support python3.9 * TST: adding unit and integration tests to MRS * MNT: use pylint flag to fix linter * TST: adding tests to MonteCarlo comparison features * MNT: applying suggestions in .rst, better handling nested variables in MRS and applying linters * MNT: removing TODO comments from monte_carlo_plots * MNT: remove useless TODO * MNT: inserting pragmas for no cover and resolving changelog conflict commit d49c40e Author: ArthurJWH <167456467+ArthurJWH@users.noreply.github.com> Date: Fri Apr 11 16:11:20 2025 -0400 ENH: Create a rocketpy file to store flight simulations (#800) * ENH: added .rpy file functionality (see issue 668) This commit add 'save_to_rpy' and 'load_from_rpy' functions, that allows saving and loading flights. * MNT: adjusting minor changes to .rpy functions and tests. Formatted docstrings correctly. Reverted duplication of `test_encoding.py` files. Version warning will be called when loaded version is more recent. * MNT: incorporating previous comments Change file management from os to Path Adjust docstrings * DOC: Added comment about outputs in `to_dict` method * MNT: Refactoring `RocketPyDecoder` unpacking operation and other small adjustments * DOC: update changelog * STY: formatted according to ruff * MNT: changing `str | Path` operation to support Python 3.9 * MNT: fixed trailing commas on .rpy and added shield against `ruff` formatting .rpy and .json files * MNT: fixing error related to `test_flight_save_load_no_resimulate` When `include_outputs` were set to `True`, it would try to include the additional data into the flight, breaking the test * MNT: fixing a typo and adding comment on test coverage --------- Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> commit 6bf70f3 Author: Júlio Machado <85506246+juliomachad0@users.noreply.github.com> Date: Sat Apr 5 15:08:53 2025 -0300 ENH: Support for the RSE file format has been added to the library (#798) * ENH: Support for the RSE file format has been added to the library. The import_rse method in the Abstract Motor class and the load_from_rse_file method in the GenericMotor class are now available. With this update, the library natively supports Rock Sim software data, eliminating the need for users to manually convert motor files. The implementation was based on the import_eng and load_from_eng_file methods, utilizing Python's standard XML library. * ENH: Adding tests to the methods of .rse file treatment. * ENH: fixing mistakes on the method and test file * MNT: Running ruff * MNT: Adding the PR to CHANGELOG.md commit 220bb59 Merge: 4a41f7a 4df0b38 Author: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Date: Thu Mar 27 06:14:22 2025 -0300 Merge pull request #797 from RocketPy-Team/master Updates develop after 1.9.0 commit 4df0b38 Author: MateusStano <69485049+MateusStano@users.noreply.github.com> Date: Mon Mar 24 17:35:03 2025 +0100 REL: Update version to 1.9.0 (#795) commit 5328d66 Author: MateusStano <69485049+MateusStano@users.noreply.github.com> Date: Mon Mar 24 13:07:52 2025 +0100 DEP: Remove Pending Deprecations and Add Warnings Where Needed (#794) * DEP: Add deprecation warnings for outdated methods and functions * DEP: Remove deprecated methods for NOAA RUC soundings and power drag plots * DEV: changelog * MNT: ruff * DEP: Update deprecation warning for post_process method to specify removal in v1.10 * MNT: Remove unused imports commit 76fb5ef Merge: a4b42c3 4a41f7a Author: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Date: Sun Mar 23 19:17:16 2025 -0300 Merge pull request #793 from RocketPy-Team/develop DEV: Master to v1.9.0 commit 4a41f7a Author: Kevin Alcañiz <kevinalcaniz22@gmail.com> Date: Sun Mar 23 21:52:51 2025 +0100 ENH: Introduce the StochasticAirBrakes class (#785) * wind factor bug corrected the wind factor wasn't applied to the env.wind_velocity properties * BUG: StochasticModel visualize attributes of a uniform distribution It showed the nominal and the standard deviation values and it doesn't make sense in a uniform distribution. In a np.random.uniform the 'nominal value' is the lower bound of the distribution, and the 'standard deviation' value is the upper bound. Now, a new condition has been added for the uniform distributions where the mean and semi range are calculated and showed. This way the visualize_attribute function will show the whole range where the random values are uniformly taken in * variable names corrections * Corrections requested by the pylint test * ENH: add multiplication for 2D functions in rocketpy.function Added the ability to multiply functions with 2D domains in the __mul__ function * ENH: StochasticAirBrakes class created The StochasticAirBrakes class has been created. The __init__.py files in the stochastic and rocketpy folders have also been modified accordingly to incorporate this new class * ENH: set_air_brakes function created This functions appends an airbrake and controller objects previuosly created to the rocket * ENH: add StochasticAirBrake to rocketpy.stochastic_rocket Some functions has been modified and other has been created in order to include the new StochasticAirBrakes feature into the StochasticRocket class. A new function named 'add_air_brakes' has been created to append a StochasticAirBrakes and Controller objects to the StochasticRocket object. A new function '_create_air_brake' has been introduced to create a sample of an AirBrake object through a StochasticAirBrake object. Enventually, the 'create_object' function has been modified to add the sampled AirBrakes to the sampled Rocket * BUG: StochasticAirBrake object input in _Controller When defining the _Controller object a StochasticAirBrake was input. This is already corrected and a AirBrake object is now introduced * ENH: add time_overshoot option to rocketpy.stochastic_flight Since the new StochasticAirBrake class is defined, we need the 'time_overshoot' option in the Flight class to ensure that the time step defined in the simulation is the controller sampling rate. The MonteCarlo class has had to be modified as well to include this option. * DOC: StochasticAirBrakes related documentation added Documentation related to the StochasticAirBrakes implementation has been added in StochasticAirBrakes, StochasticRocket and Rocket classes. * ENH: pylint recommendations done * ENH: Reformatted files to pass Ruff linting checks * ENH: Update rocketpy/stochastic/stochastic_rocket.py Unnecessary comment Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> * DOC: improve drag curve factor definition in StochasticAirBrakes * ENH: Change assert statement to if Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> * DOC: better explanation of __mul__ function Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> * ENH: delete set_air_brakes function for simplicity * DOC: CHANGELOG file updated --------- Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> commit 90553f5 Author: Kevin Alcañiz <kevinalcaniz22@gmail.com> Date: Sun Mar 23 20:31:50 2025 +0100 ENH: Add Eccentricity to Stochastic Simulations (#792) * wind factor bug corrected the wind factor wasn't applied to the env.wind_velocity properties * BUG: StochasticModel visualize attributes of a uniform distribution It showed the nominal and the standard deviation values and it doesn't make sense in a uniform distribution. In a np.random.uniform the 'nominal value' is the lower bound of the distribution, and the 'standard deviation' value is the upper bound. Now, a new condition has been added for the uniform distributions where the mean and semi range are calculated and showed. This way the visualize_attribute function will show the whole range where the random values are uniformly taken in * variable names corrections * Corrections requested by the pylint test * ENH: more intuitive uniform distribution display in StochasticModel Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> * ENH: Eccentricities added to the StochasticRocket class A bug has been corrected in Flight class and an enhancement has been performed in the Rocket class as well * BUG: thrust eccentricity bug corrected eccentricity_y was defined by x coordinate and eccentricity_x was defined by y coordinate * BUG: Undo some Rocket class changes * ENH: add eccentricities to StochasticRocket * BUG: fix MonteCarlo eccentricity inputs * ENH: pylint and ruff recommended changes * TST: fix tests with eccentricity --------- Co-authored-by: Gui-FernandesBR <guilherme_fernandes@usp.br> commit 7348053 Author: Kevin Alcañiz <kevinalcaniz22@gmail.com> Date: Sun Mar 23 13:49:35 2025 +0100 BUG: fix the wind velocity factors usage and better visualization of uniform distributions in Stochastic Classes (#783) * wind factor bug corrected the wind factor wasn't applied to the env.wind_velocity properties * BUG: StochasticModel visualize attributes of a uniform distribution It showed the nominal and the standard deviation values and it doesn't make sense in a uniform distribution. In a np.random.uniform the 'nominal value' is the lower bound of the distribution, and the 'standard deviation' value is the upper bound. Now, a new condition has been added for the uniform distributions where the mean and semi range are calculated and showed. This way the visualize_attribute function will show the whole range where the random values are uniformly taken in * variable names corrections * Corrections requested by the pylint test * ENH: more intuitive uniform distribution display in StochasticModel Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> --------- Co-authored-by: MateusStano <69485049+MateusStano@users.noreply.github.com> commit d2f89ba Author: Leonardo Rosa <leogabriel3@gmail.com> Date: Fri Mar 21 18:57:49 2025 -0300 DEV: add requirements-tests.txt on make install target (#791) * DEV: adds 'pip install -r requirements-tests.txt' recipe to 'make install' target on Makefile Co-authored-by: Gui-FernandesBR <63590233+Gui-FernandesBR@users.noreply.github.com> commit 91ac567 Author: Leonardo Rosa <leogabriel3@gmail.com> Date: Fri Mar 21 18:53:53 2025 -0300 BUG: fixes get_instance_attributes for Flight objects containing a Rocket object without rail buttons (#786) * DOC: fixed a typo in funcify_method() description * TST: created test for get_instante_attributes() with flight without rail buttons * BUG: fixed __calculate_rail_button_forces() by assigning a Function(0) to null_force instead of an empty array * DEV: updates CHANGELOG commit 9407470 Author: Leonard <74966503+L30-stack@users.noreply.github.com> Date: Wed Mar 19 16:01:59 2025 +0100 BUG: fixed AGL altitude in _FlightPrints.events_registered (#788) * BUG: fixed AGL altitude in _FlightPrints.events_registered * updeted CHANGELOG
7244
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.