Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ENH: Create a dataset of pre-registered motors. #664

Open
Gui-FernandesBR opened this issue Aug 19, 2024 · 0 comments · Fixed by #744
Open

ENH: Create a dataset of pre-registered motors. #664

Gui-FernandesBR opened this issue Aug 19, 2024 · 0 comments · Fixed by #744
Labels
Enhancement New feature or request, including adjustments in current codes Good first issue Good for newcomers Motors Every propulsion related issue or PR

Comments

@Gui-FernandesBR
Copy link
Member

Gui-FernandesBR commented Aug 19, 2024

Something OpenRocket does pretty well is to provide a wide range of available motors already on the software.
Maybe we could do something similar with .json files.

There are 2 options for us to accomplish this issue:

  1. Save .json files with all the information we may find available on internet.
  2. Download and save several .eng files in the repo so we can install it along with the rocketpy package itself.

There's already na open PR solving this kind of issue in the first rocketpy hackathon (2020)

@Gui-FernandesBR Gui-FernandesBR added Enhancement New feature or request, including adjustments in current codes Good first issue Good for newcomers Motors Every propulsion related issue or PR labels Aug 19, 2024
caioessouza added a commit to caioessouza/RocketPy that referenced this issue Dec 2, 2024
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!
Gui-FernandesBR pushed a commit to caioessouza/RocketPy that referenced this issue Dec 8, 2024
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
Gui-FernandesBR added a commit that referenced this issue Dec 8, 2024
* 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 <[email protected]>
@Gui-FernandesBR Gui-FernandesBR linked a pull request Dec 8, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request, including adjustments in current codes Good first issue Good for newcomers Motors Every propulsion related issue or PR
Projects
Status: Backlog
Development

Successfully merging a pull request may close this issue.

1 participant