Skip to content
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

Clean up parameter handling in forecasting pipeline #33

Merged
merged 1 commit into from
Apr 5, 2024

Conversation

wgifford
Copy link
Collaborator

@wgifford wgifford commented Apr 5, 2024

  • Cleans up defaults in init, using a similar approach to the audio pipelines in HF.
  • Adds an option to control inverse scaling of outputs (driven by the time series preprocessor)
  • Fixes some type hints

To do (possibly in a subsequent PR):
Better integration of the time series preprocessor in the pipeline.
User shouldn't have to pass all the parameters for columns multiple times, etc.
Use the preprocessor in the pipeline to actually preprocess and scale data!

Signed-off-by: Wesley M. Gifford <[email protected]>
@wgifford wgifford merged commit 64eb44a into main Apr 5, 2024
4 checks passed
@wgifford wgifford deleted the pipeline_updates branch April 5, 2024 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant