neuralforecast icon indicating copy to clipboard operation
neuralforecast copied to clipboard

[FIX] 926 auto ensure all models support alias and 924 Configuring hyperparameter space for Auto* Models

Open elephaint opened this issue 11 months ago • 2 comments

Fixes #926 and fixes #924.

  • Fix of #926: Fixed Auto* models that do not support all required variables from BaseAuto Add unit test to all Auto* models to test if the Auto* models contain all required variables from BaseAuto Some reformatting of models.ipynb, putting unit tests per model in hidden cell below each model
  • Fix of #924: Reformulated a part of the Ray-portion of the Automated Hyperparameter tuning guide, explaining how users can access default settings for hyperparameters.

elephaint avatar Mar 13 '24 14:03 elephaint

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

LGTM! Let's just please also wait for @cchallu's review as well.

Thanks - it does not fix the default_config issue yet, so I'm changing a few things (converted to draft)

elephaint avatar Mar 14 '24 18:03 elephaint

  • Added a get_default_config method for all Auto* models
  • Fixed incompatible hyperparameter configurations in base models
  • Added unit tests to all auto models to ensure they all run with default configs, on both backends.
  • Update of Hyperparameter tuning guide to demonstrate the use of get_default_config

elephaint avatar Mar 15 '24 15:03 elephaint