-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
[ENH] Add support for Windows in TimesFMForecastor #8291
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
Comments
sktime
, I can use times-fm
without any problem e.g. using something like
Hey, can i please be assigned this issue? I would like to work on it . |
i think this issue is not related to this repo. |
Hey @viditjain44 ! I would like to say that it is related to this repo, as sktime is a toolbox and it's made to support various forecasters and give a unified interface for them. I say so because timesfm 2.x the pytorch implementation supports python versions above 11.0 and I think that it should also be supported in sktime. I believe it's a new feature that can be worked out. Maybe as a new Forecastor?? |
Hey @4x! I think you can use the following parameters and check if it works once.. ignore_deps : bool, default=False
If True, dependency checks will be ignored, and the user is expected to handle
the installation of required packages manually. If False, the class will enforce
the default dependencies required for the vendor library or the pypi
package, as described above, via ``use_source_package``. I hope it works out! |
Hi @OmBiradar - thank you for the suggestion! Unfortunately, I am still getting the same |
Dear @viditjain44 - I'm not sure why you think this is not relate to this repo?
@OmBiradar: there already is a Thank you both! |
Hey Anyone willing to contribute please look into this comment and the whole issue in general to get a sense of why it's made that way. I think the main issue is that TimesFM is not stable on many sytems, thus the highly restrictive nature in sktime. I believe after many monts it might have improved and let's look into what has changed and what can be used! |
Hey @fkiraly! Should we try contacting the authors of the TimesFM forecast wrapper in sktime? I guess they can provide the valuable input? Also, I guess you were also involved in the forecaster.... can you help? |
Hello everyone i would like to say that i think for resolving this issue we have first add some configuration in the timesfm repo (https://github.com/google-research/timesfm) . I am saying this because i have tried to resolve it. Also i am open to discussion and speaking till my knowledge . It is so possible that i am wrong . |
The forecaster is currently using a fork in the I would suggest to remove the tag and check the different ways in which this could be solved:
|
I use Python 3.13 in Windows 11. Outside of
sktime
, I can usetimes-fm
without any problem e.g. using something likeHowever, within
sktime
, theTimesFMForecaster
requires (a) to downgrade to 3.10 and (b) to use... Linux:Could we kindly request to remove/relax these requirements? Sure, I can spin up a 3.10 venv but Linux?!
Thank you
Originally posted by @4x in #7138 (comment)
The text was updated successfully, but these errors were encountered: