-
Notifications
You must be signed in to change notification settings - Fork 6
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
OS X crash on import paseos when pip installed #51
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor changes
README.md
Outdated
@@ -7,7 +7,7 @@ This project is currently under development. Use at your own risk. :) | |||
|
|||
<p align="center"> | |||
<a href="https://github.com/aidotse/PASEOS/"> | |||
<img src="resources/images/datastructure.svg" alt="Scheme" width="910" height="459"> | |||
<img src="paseos/resources/images/datastructure.svg" alt="Scheme" width="910" height="459"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's keep the readme images (this and below 2) in the top-level folder resources, there is no need to ship them with the module and otherwise they have no reason to be in the module folder
@@ -140,6 +140,6 @@ async def job(): | |||
return job() | |||
else: | |||
# Run activity and processor | |||
asyncio.run(job()) | |||
asyncio.gather(job()) | |||
|
|||
logger.info(f"Activity {activity} completed.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pls remove the file paseos/resources/.DS_Store
|
||
|
||
def test_activity(): | ||
@pytest.mark.asyncio |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a comment somewhere what this actually does, so we remember? :)
paseos/utils/load_default_cfg.py
Outdated
) | ||
logger.debug(f"loading path: {path}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
logger.debug(f"loading path: {path}") | |
logger.debug(f"Loading default cfg from {path}") |
(can then remove above debugger output in Line9)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Description
Summary of changes
Resolved Issues
How Has This Been Tested?
Related Pull Requests