Skip to content
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

Change SeasonalNaive fallback predictor to nanmean #2549

Merged
merged 2 commits into from
Jan 11, 2023

Conversation

abdulfatir
Copy link
Contributor

Issue #, if available:

Description of changes: Changed SeasonalNaive prediction from mean to nanmean in case time series is shorter than season_length.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Please tag this pr with at least one of these labels to make our release process faster: BREAKING, new feature, bug fix, other change, dev setup

@lostella
Copy link
Contributor

Could this be considered a bugfix?

@abdulfatir
Copy link
Contributor Author

I would say so.

@lostella lostella added bug fix (one of pr required labels) pending v0.11.x backport This contains a fix to be backported to the v0.11.x branch labels Jan 11, 2023
@lostella lostella enabled auto-merge (squash) January 11, 2023 13:24
@lostella lostella merged commit cdeab67 into awslabs:dev Jan 11, 2023
@abdulfatir abdulfatir deleted the seasonal-fix branch January 11, 2023 14:05
lostella pushed a commit to lostella/gluonts that referenced this pull request Jan 17, 2023
@lostella lostella mentioned this pull request Jan 17, 2023
lostella added a commit that referenced this pull request Jan 17, 2023
* Update workflow actions to latest versions (#2447)

* Simplify setup.py (#2525)

* Use Justfile for checks.

* Fix dataclass eventual handling. (#2530)

* Improve README example (#2538)

* Change SeasonalNaive fallback predictor to nanmean (#2549)

Co-authored-by: Abdul Fatir Ansari <[email protected]>

* Compact PandasDataset string representation (#2558)

Co-authored-by: Jasper <[email protected]>

Co-authored-by: Lorenzo Stella <[email protected]>

* Update `MANIFEST.in`. (#2566)

Fixes: #2564

We want to distribute more files in the source distribution,
such that GluonTS can be more easily be imported in other
distributions, e.g. conda.

* Add NaN validation to Evaluator (#2568)

Co-authored-by: Jasper <[email protected]>
Co-authored-by: Abdul Fatir <[email protected]>
Co-authored-by: Abdul Fatir Ansari <[email protected]>
Co-authored-by: Lorenzo Stella <[email protected]>
@lostella lostella removed the pending v0.11.x backport This contains a fix to be backported to the v0.11.x branch label Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix (one of pr required labels)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants