Allow custom actor properties

[gomezzz]

Description

Summary of changes

• Added ability for custom properties
• Added automatic monitoring of them
• Added new test for custom properties
• Small refactoring, reformatting (did we want 100? 127? flake has 127 but that is actually a lot 🤔 )

Resolved Issues

N/A

How Has This Been Tested?

CI/CD, new test

Related Pull Requests

N/A

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
paseos
__init__.py	31	2	94%	46, 49
paseos.py	151	14	91%	71–72, 144–145, 198, 213, 231, 240, 258, 289, 297–298, 398–399
paseos/activities
activity_manager.py	43	3	93%	46, 76, 159
activity_processor.py	58	1	98%	123
activity_runner.py	62	10	84%	80–84, 104–113, 122–125
paseos/actors
actor_builder.py	108	21	81%	20–24, 27, 119–124, 248, 296, 318, 328–332, 339–340, 345–346
base_actor.py	121	18	85%	82, 99, 115, 153, 193–195, 224, 233, 251, 268, 272–275, 279, 293, 335
ground_station_actor.py	15	3	80%	47–53
spacecraft_actor.py	61	1	98%	123
paseos/communication
get_communication_window.py	22	2	91%	38, 67
is_in_line_of_sight.py	74	15	80%	75, 82, 90, 152–169, 213, 233
paseos/power
charge_model.py	17	1	94%	54
discharge_model.py	7	2	71%	22, 34
is_in_eclipse.py	39	4	90%	28–29, 90–91
paseos/tests
activity_test.py	57	3	95%	92, 95–96
communication_window_test.py	58	1	98%	198
eclipse_test.py	11	1	91%	22
import_test.py	6	1	83%	13
init_test.py	8	1	88%	16
line_of_sight_test.py	65	4	94%	147–150
thermal_model_test.py	30	1	97%	63
visualization_test.py	20	1	95%	31
paseos/utils
check_cfg.py	61	18	70%	31, 40, 46, 62–64, 71, 74–76, 79–81, 84–86, 101, 106
operations_monitor.py	71	4	94%	88, 129–132
paseos/visualization
animation.py	18	6	67%	27–30, 35, 44
plot.py	9	3	67%	28–31
space_animation.py	195	20	90%	135, 159, 162–165, 171, 283, 308–310, 315, 325–326, 330, 365, 371–372, 405, 421–432
TOTAL	1831	161	91%

Tests	Skipped	Failures	Errors	Time
30	0 💤	0 ❌	0 🔥	1m 4s ⏱️

[GabrieleMeoni]

All good for me except one very minor request. I tested it, and it seems working.
About flake I would leave 127 chars, otherwise it becomes very annoying.

[gomezzz]

@GabrieleMeoni

About flake I would leave 127 chars, otherwise it becomes very annoying.

But then we should reformat because before we agreed on 100 and most of the repo is in 100 but now some parts are 127. So you will have a lot of formatting changes either ways because it is now inconsistent.

I would suggest to create an issue to either:

• format repo to 127
• format to 100 and change flake