Conversation
|
AllenBaranov seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
sarahmish
left a comment
There was a problem hiding this comment.
@abaranov25 seems like Chronos2 pipeline did not work well on S-1 example, do you have an idea why? I'm curious if you use S-1-test would you get different results?
There was a problem hiding this comment.
I'm curious why 256? Orion pipelines that are forecast based are usually set to 250. For timesfm, it was set to 256 because it needs to be a multiple of 32 according to their docs.
There was a problem hiding this comment.
We typically set the example to interval=21600 since our example code uses NASA signals (MSL and SMAP)
orion/primitives/chronos2.py
Outdated
There was a problem hiding this comment.
If chronos works on python 3.9+ remove this comment completely
orion/primitives/chronos2.py
Outdated
There was a problem hiding this comment.
add chronos to setup.py under the pretrained_requires section, then remove the try-catch since it will always be installed using pip install orion[pretrained]
orion/primitives/chronos2.py
Outdated
sarahmish
left a comment
There was a problem hiding this comment.
@abaranov25 It would be nice to run the notebook locally then commit it so that the output is rendered into the notebook , then users can look at the github and immediately see the results of Chronos 2
| "\n", | ||
| "MLPipelines are compose of a squence of primitives, these primitives apply tranformation and calculation operations to the data and updates the variables within the pipeline. To view the primitives used by the pipeline, we access its `primtivies` attribute. \n", | ||
| "\n", | ||
| "The `UniTS` contains 6 primitives. we will observe how the `context` (which are the variables held within the pipeline) are updated after the execution of each primitive." |
There was a problem hiding this comment.
This line needs to be updated
Resolve #662
Adding Chronos-2 as a new pipeline on Orion