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.

Sign up for GitHub

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

Jump to bottom

Continuous Integration Tests #130

Closed
kdgutier opened this issue Dec 14, 2022 · 1 comment · Fixed by #129 or #141
Closed

Continuous Integration Tests #130

kdgutier opened this issue Dec 14, 2022 · 1 comment · Fixed by #129 or #141
Assignees
@AzulGarza @kdgutier @cchallu

Comments

@kdgutier
Copy link
Collaborator

It would be convenient to add continuous integration tests to monitor performance and fitting times of the reconciliation methods.
Automated tests will hasten the development and improvements of the code.
@kdgutier kdgutier linked a pull request Jan 4, 2023 that will close this issue
CircleCI #129
Merged
@kdgutier kdgutier linked a pull request Jan 4, 2023 that will close this issue
[FIX] Circle ci integration #141
Merged
@kdgutier
Copy link
Collaborator Author

kdgutier commented Jan 4, 2023

In the future I would like to change the current CI test towards:

  • Medium sized dataset TourismL with strictly hierarchical and non-strictly hierarchical version.
  • sCRPS/MSSE measurements.
  • Improved execution time recordings.
  • Pre computed AutoARIMA predictions for the tests to run fast.

I opened a new issue with these comments.

@kdgutier kdgutier closed this as completed Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AzulGarza AzulGarza

@kdgutier kdgutier

@cchallu cchallu

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

CircleCI Nixtla/hierarchicalforecast
[FIX] Circle ci integration Nixtla/hierarchicalforecast
3 participants
@AzulGarza @kdgutier @cchallu