Skip to content

Refactoring of the forecast tutorial#1070

Closed
emanuel-schmid wants to merge 5 commits into
developfrom
feature/refactoring_forecast_tutorial
Closed

Refactoring of the forecast tutorial#1070
emanuel-schmid wants to merge 5 commits into
developfrom
feature/refactoring_forecast_tutorial

Conversation

@emanuel-schmid
Copy link
Copy Markdown
Collaborator

Changes proposed in this PR:

This PR fixes #

PR Author Checklist

PR Reviewer Checklist

Copy link
Copy Markdown
Member

@chahank chahank left a comment

Choose a reason for hiding this comment

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

It looks like together with the typo correction, all the plot have been regenerated. This creates quite a lot of changes in the git log. Do we want this?

Otherwise good to go. I think neither the pre-commit hook fnor petals compatibility failing is related to this PR.

@emanuel-schmid
Copy link
Copy Markdown
Collaborator Author

emanuel-schmid commented Jul 9, 2025

@chahank yeah. right. The problem is that every plot change adds to the size of the repository in the range of MBytes.
In the long run this leads to a download size for the whole repository that is just not bearable anymore.

@elianekobler - I know that changing only the relevant lines in a notebook is a bit tricky, but still ...

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants