-
Notifications
You must be signed in to change notification settings - Fork 780
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
PatchTSMixer tutorial broken. ForecastDFDataset API change #2120
Comments
We have deprecated ForecastDataset API and using TSP for dataset generation. you can check the example here https://colab.research.google.com/github/IBM/tsfm/blob/tutorial/notebooks/tutorial/ttm_tutorial.ipynb Also, you can also try using TTM (which is an advanced version of PatchTSMixer). Here is the model card https://huggingface.co/ibm-granite/granite-timeseries-ttm-v1 |
I am trying to use the regular PatchTST model per this documentation :https://huggingface.co/blog/patchtst. I run into the same problems as the original issue and have tried the links provided as a solution, however that had more issues. Any insight on a fix? |
cc @kashif |
Same problem here... |
thanks for the report @etpereira let me have a look again |
@kashif , I was just looking at this link https://github.com/ibm-granite/granite-tsfm/blob/main/notebooks/hfdemo/patch_tsmixer_getting_started.ipynb And discover a mode of doing it work: The correct arguments are:
Thank you, anyway. |
in the tutorial this is how ForecastDFDataset is used:
train_dataset = ForecastDFDataset(
time_series_processor.preprocess(train_data),
id_columns=id_columns,
timestamp_column="date",
input_columns=forecast_columns,
output_columns=forecast_columns,
context_length=context_length,
prediction_length=forecast_horizon,
)
but input_columns and output_columns are no longer valid parameters. What is the correct usage?
The text was updated successfully, but these errors were encountered: