-
Notifications
You must be signed in to change notification settings - Fork 8
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
Update Model Energg Demand and Fix Problem of Submodels #55
Update Model Energg Demand and Fix Problem of Submodels #55
Conversation
Hi Emre!!! Then locally by updating the environment things work, I thought the changes were enough to change the CI-Linux enviroment as well, but that was not the case! Theoretically I think the problem may be on the version of the Fiona package, after the Pypsa-Earth updates. |
@davide-f can you please check with this morning's change which environment is used by CI-Linux? |
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.
Few comments :) many thanks!
Indeed, the CI is not using the new env yet. With the proposed comment, it should work out
Co-authored-by: Davide Fioriti <[email protected]>
…ta-Capitani/pypsa-distribution into Demand_Update_Modeling_
for more information, see https://pre-commit.ci
…ta-Capitani/pypsa-distribution into Demand_Update_Modeling_
for more information, see https://pre-commit.ci
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.
Great @Margherita-Capitani :D
Merging
In this pull request, the following are proposed:
Updates have been tested locally and are working. Updating the environment requires recreating the environment locally.