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

CI/CD update: successful cosim_example.py execution from built package & fixed file path issues #125

Closed
wants to merge 2 commits into from

Conversation

amandamalk0601
Copy link
Contributor

Updated the CI/CD pipeline to:

  • Use Poetry to build and install a package from the latest main branch code.
  • Test the installed package by running the example scenario cosim_example.py, verifying file paths are set correctly.

Changes made:

  • Modified pyproject.toml to include examples/ and tests/ when building the package.
  • Fixed file path issues in _data.py to correctly locate the solar and carbon data files when executing example scenario from the installed package.

The file path issues were resolved using the files function from importlib.resources, which is natively available in Python from version 3.9 onwards. For compatibility with Python 3.8, the external importlib_resources library was added to pyproject.toml.

@amandamalk0601 amandamalk0601 added the enhancement New feature or request label Aug 25, 2023
@amandamalk0601 amandamalk0601 self-assigned this Aug 25, 2023
Copy link
Contributor

@marvin-steinke marvin-steinke left a comment

Choose a reason for hiding this comment

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

cool beans

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants