Skip to content

added a configuration file and library #62

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.

Sign up for GitHub

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

Jump to bottom
Open
waridh wants to merge 6 commits into
base: main
Choose a base branch
from
Open

added a configuration file and library #62

waridh wants to merge 6 commits into from

Conversation

@waridh
Copy link
Contributor

@waridh waridh commented Jul 14, 2024 •
edited
Loading

#44 #61

Included

  • A Python helper library that takes the path to the config file, and the subsystem of interest, and returns a dictionary with the settings needed. For now, the only field is the port.
  • A configuration file of the .ini format. Chosen for ease of modification.
  • A definitions.py that contains immutable shared values. Most notable is a dynamic absolute path of the project root (It will be the correct location on any machine thanks to pathlib.Path).
  • Simple Pytest tests for this helper library.
  • Basic documentation on how to use the microlibrary, and write new pytest tests.

@waridh
added a configuration file and library
d0b39bc 
The simulated subsystems are now able to grab their configuration from
the project root (simulated_config.ini)
@waridh waridh added the enhancement New feature or request label Jul 14, 2024
@waridh waridh self-assigned this Jul 14, 2024
@waridh waridh linked an issue Jul 14, 2024 that may be closed by this pull request
Single file in project root directory with mention of each port in use and a description of its use. #61
Open
4 tasks
@waridh waridh requested review from jac4e and kulovac July 16, 2024 22:40
dheadrick1618
dheadrick1618 requested changes Aug 13, 2024
View reviewed changes
Copy link
Member

@dheadrick1618 dheadrick1618 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! This is a good starting point for us moving forward with testing the sim subsystems.

System tests however will be tricky with the OBC FSW interacting with the simulated subsystems.

@waridh
Copy link
Contributor Author

waridh commented Sep 6, 2025

@rrasmuss4200 @kulovac, guys, what is the status of the changes requested by this branch? Has the work that would have been added by this branch been merged into main?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dheadrick1618 dheadrick1618 dheadrick1618 requested changes

@rcunrau rcunrau Awaiting requested review from rcunrau

@rrasmuss4200 rrasmuss4200 Awaiting requested review from rrasmuss4200

@jac4e jac4e Awaiting requested review from jac4e

@kulovac kulovac Awaiting requested review from kulovac

Requested changes must be addressed to merge this pull request.

Assignees

@waridh waridh

Labels

enhancement New feature or request

Projects

Ex3 Simulated Satellite System Project Management
Status: Requires Review

Milestone

Simulated subsystems roughly complete

Development

Successfully merging this pull request may close these issues.

Single file in project root directory with mention of each port in use and a description of its use.

3 participants

@waridh @dheadrick1618