Anirban Ray
Anirban Ray
FYI @pranavvp16 - this is the first draft, and you can take a look and try it out. I am facing failures with almost any change I try with `AutoTFT`...
@geetu040 can you please confirm if you tested the `freq` being changed to optional parameter? CI is failing with the following errors already, so wondering why CI in your PR...
@jmoralez can you please tell if these failures are caused by https://github.com/Nixtla/neuralforecast/pull/833 or not? If that's the case, I think https://github.com/sktime/sktime/pull/6039 to make `freq` optional may need to be revisited.
> we should change this particular test case test_neural_forecast_fail_with_auto_freq_on_range_index to adopt with neuralforecast==1.7.0 Will that only not satisfy a pass CI? I think we do need to change the class....
The new filter feature is awesome, thank you for this contribution. Few feedbacks: 1. Not sure if it's due to some extensions or not, but loading estimator overview page is...
> > Tags rename? Remove unneeded tags? > > Need feedback from maintainers. I have no comments, requesting @fkiraly / @benHeid to pitch in. --- > > For others, I...
FYI @duydl this may be something of your interest
Is there an way to automatically create issues from failure on `main` branch? If not, may be we should track this in an issue (potentially good first issue).
I just commented in Discord as well, this job is not coming from "other" case in new CI. It's coming from old CI relying on the Makefile, specifically this line...
I've created PR #6353, please take a look. But based on sample runs on my laptop, this is going to make new CI significantly slower, even the test collection part....