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

Student clean #204

Merged
merged 11 commits into from
Feb 1, 2024
Merged

Student clean #204

merged 11 commits into from
Feb 1, 2024

Conversation

GabrieleMeoni
Copy link
Collaborator

Description

Summary of changes

  • Cleaning students'code.
  • Restyling code to be compliant with PASEOS

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link

github-actions bot commented Feb 1, 2024

Overall Coverage

Coverage Report
FileStmtsMissCoverMissing
paseos
   __init__.py33197%49
   paseos.py1471292%67–68, 145–146, 211, 226, 244, 253, 271, 302, 310–313
paseos/activities
   activity_manager.py43393%46, 74, 157
   activity_processor.py58198%121
   activity_runner.py621084%80–84, 104–113, 122–125
paseos/actors
   actor_builder.py2073185%24–30, 33, 223–225, 252–254, 299–308, 338, 344, 492, 568, 602, 624, 655, 667–671, 678–679, 686–687
   base_actor.py1351887%85, 107, 123, 188, 228–230, 251, 260, 277, 298, 304–307, 313, 328, 383
   ground_station_actor.py15380%47–53
   spacecraft_actor.py92595%166, 221, 230–233
   spacecraft_body_model.py30390%100, 113, 122
paseos/attitude
   attitude_model.py71199%124
paseos/central_body
   central_body.py74791%70, 179–180, 188–189, 258–260
   is_in_line_of_sight.py581279%106–123, 167, 187
   mesh_between_points.py35294%71, 79
   sphere_between_points.py26869%35–38, 48–51, 63–64
paseos/communication
   get_communication_window.py23291%38, 64
paseos/power
   charge_model.py16194%49
   discharge_model.py7271%22, 34
paseos/tests
   activity_test.py57395%90, 93–94
   attitude_test.py683351%93–129, 137–166
   communication_window_test.py49198%176
   eclipse_test.py10190%20
   import_test.py6183%13
   init_test.py8188%16
   line_of_sight_test.py62494%143–146
   mesh_test.py119397%118, 139, 147
   thermal_model_test.py30197%61
   visualization_test.py18194%28
paseos/utils
   check_cfg.py611870%31, 40, 46, 60–62, 67, 70–72, 75–77, 80–82, 98, 103
   operations_monitor.py71396%127–130
   reference_frame_transfer.py67888%108, 132, 240–242, 245–247
paseos/visualization
   animation.py18667%27–30, 35, 44
   plot.py9367%29–32
   space_animation.py2152389%108–109, 124, 153–154, 188, 212–219, 225, 335, 358–360, 365, 374–375, 379, 415, 421–422, 453, 467–478
TOTAL266223291% 

Tests Skipped Failures Errors Time
46 0 💤 0 ❌ 0 🔥 1m 8s ⏱️

@GabrieleMeoni GabrieleMeoni merged commit ef1d0d7 into student Feb 1, 2024
3 checks passed
@GabrieleMeoni GabrieleMeoni deleted the student_clean branch February 1, 2024 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant