Conversation
|
Thanks Francesco, good Idea! I will test the configs and let you know. |
Louis-Frey
left a comment
There was a problem hiding this comment.
So I tested all the configs, with the exception of forecasters-co1e.yaml (this was expected to fail), and before main was merged into this branch.
forecasters-ich1-oper-fixed.yaml produces some errors when initially run on the 2024 cases, but when I run it first on the 2025 case, and then on the 2024 cases again, it runs fine. I will take it low-prio on my list to look into that as well.
The other configs run without error. Good to go from my side.
Since this is the config that we use for the latest experiments, I would not merge this until this is fixed. What was the error? |
This PR introduces guidelines for extending the evalml workflow and enforces them. It's almost entirely renaming some rules and files so it should not break anything, but I would test anyway.