-
Notifications
You must be signed in to change notification settings - Fork 0
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
Features/#36 update airflow #111
Conversation
Features/mail notifications
…flow-providers-sendgrid
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.
During the installation in a new virtual environment, there were problems with the package psycopg2. Upgrading setuptools from 59.6.0 to 67.8.0 solved the problem. After this, the installation was successful.
When executing egon-data serve, the following problem was found:
AttributeError: module 'importlib_resources' has no attribute 'read_text'
This was solved by downgrading importlib-sources from 6.0.1 to 5.12.0.
Could be worth including a comment about the package docker-compose in the installation instructions: It must be installed using pip when it is not installed globally.
if ( | ||
"eGon2035" in egon.data.config.settings()["egon-data"]["--scenarios"] | ||
) and not ( | ||
"status2019" in egon.data.config.settings()["egon-data"]["--scenarios"] | ||
): | ||
tasks = tasks + (insert_data_nep,) | ||
|
||
if "eGon100RE" in egon.data.config.settings()["egon-data"]["--scenarios"]: |
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.
Does it mean that for eGon100RE the task "insert_data_nep" should not be executed?
The problem with the username and password when opening the airflow web interface should be fixed now. |
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.
Once the comments about dependencies are addressed, this PR can be merged.
Adding |
Fixes #36 .