Skip to content
This repository has been archived by the owner on Oct 30, 2022. It is now read-only.

Issue #16: team-05 - Dispersion class #106

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

Sofia-ME
Copy link

Pull request type

Please check the type of change your PR introduces:

  • [X ] Code base additions (bugfix, features)
  • Code maintenance (refactoring, formatting, renaming, tests)
  • ReadMe, Docs and GitHub maintenance
  • Other (please describe):

Pull request checklist

Please check if your PR fulfills the following requirements, depending on the type of PR:

  • ReadMe, Docs and GitHub maintenance:

    • Spelling has been verified
    • Code docs are working correctly
  • Code base maintenance (refactoring, formatting, renaming):

    • Docs have been reviewed and added / updated if needed
    • Lint (black rocketpy) has passed locally and any fixes were made
    • All tests (pytest --runslow) have passed locally
  • Code base additions (for bug fixes / features):

    • Tests for the changes have been added
    • Docs have been reviewed and added / updated if needed
    • Lint (black rocketpy) has passed locally and any fixes were made
    • All tests (pytest --runslow) have passed locally

What is the current behavior?

Enter text here...

What is the new behavior?

Dispersion class

Does this introduce a breaking change?

  • Yes
  • No

Other information

Enter text here...

@Gui-FernandesBR Gui-FernandesBR linked an issue Aug 28, 2022 that may be closed by this pull request
@MateusStano MateusStano added 50 points Hard challengs! and removed 50 points Hard challengs! labels Aug 28, 2022
@Gui-FernandesBR Gui-FernandesBR changed the title Issue#16_Team05 Issue #16: team-05 - Dispersion class Aug 28, 2022
@MateusStano
Copy link
Member

Hey. This is really great!
There seem to be some issues still with some of parachutes' plots/attributes and some plot formatting. Since these are minor things I will give you the 50 points, but if you still want to work on this it would be great

@MateusStano MateusStano added the 50 points Hard challengs! label Aug 28, 2022
@Gui-FernandesBR Gui-FernandesBR requested review from MateusStano and removed request for giovaniceotto August 28, 2022 22:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
50 points Hard challengs! team-05
Projects
Development

Successfully merging this pull request may close these issues.

Dispersion Analysis Class
4 participants