:hammer: Refactor manual_control

[MathisFederico]

Description

A simple refactoring of the manual_control.py file using a new ManualControl object.

Type of change

Please delete options that are not relevant.

• [X] New feature (non-breaking change which adds functionality)

Screenshots

Please attach before and after screenshots of the change if applicable.

Checklist:

• [X] I have run the pre-commit checks with pre-commit run --all-files (see CONTRIBUTING.md instructions to set it up)
• [X] I have commented my code, particularly in hard-to-understand areas
• [ ] I have made corresponding changes to the documentation
• [X] My changes generate no new warnings
• [X] I have added tests that prove my fix is effective or that my feature works
• [X] New and existing unit tests pass locally with my changes

[MathisFederico]

I'm not sure how to check if I need to modify anything in the documentation

[rodrigodelazcano]

@MathisFederico thank you, this looks great. Can you add the pytest_mock dependency under test_requirements.txt.

I think this is the cause of the error in the CI.

[MathisFederico]

@rodrigodelazcano I had put it in test_requirements.txt but the setup.py file was badly configured and did not read it ! You might want to switch to pyproject.toml config instead which is the new standard for python packages.

[rodrigodelazcano]

@MathisFederico our plan is to transition to pyproject.toml in the future, when we can assure that python developers and users have done a complete transition to it. However, if you wish to make an initial PR for future merging that would be great

[rodrigodelazcano]

Hey @MathisFederico . How are the reviewed changes going? :)

[MathisFederico]

Hey @rodrigodelazcano done !

[rodrigodelazcano]

@MathisFederico I think we should remove the extra environment pickable test. Last PR you made already had a great test for this. Can you remove this and also rebase your branch please?