google-research / timesfm

TimesFM (Time Series Foundation Model) is a pretrained time-series foundation model developed by Google Research for time-series forecasting.
https://research.google/blog/a-decoder-only-foundation-model-for-time-series-forecasting/
Apache License 2.0
3.02k stars 228 forks source link

Revert "Fix importing patch_decoder" #44

Open w-e-w opened 1 month ago

w-e-w commented 1 month ago

I'm pretty sure that "fix" actually broke this repo as a module

I belive the error ImportError: attempted relative import with no known parent package occurs if people try to run src/timesfm.py directly and they might have reason to do so because this reop did not work as a module on it's initial release

adding to this this repo is structured weirdly so people might be confused on how they should be using this repo and running the code


timeline


├── README.md
├── timesfm/            <---- package name
│   ├── __init__.py
│   ├── module.py
│   └── module.py
└── other stuff/
    ├── generate_awesomeness.py
    └── decrease_world_suck.py


the reason I don't perform the restructure myself in this PR is because a the restructure of the project would cause existing users to have issues, my intention is to only get back to a code working state

google-cla[bot] commented 1 month ago

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

w-e-w commented 1 month ago

CLA... just treat this as an issue post