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

Added MS availability to augment_calendar_df function #506

Merged
merged 2 commits into from
Apr 6, 2023

Conversation

kdgutier
Copy link
Collaborator

@kdgutier kdgutier commented Apr 5, 2023

I added the MS frequency to the augment_calendar_df utils function.
For hierarchical forecast purposes. In the future we might want to increase the available catalog in a more organized way.

@kdgutier kdgutier requested a review from cchallu April 5, 2023 22:04
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@kdgutier kdgutier merged commit 5b2bb98 into main Apr 6, 2023
8 checks passed
@kdgutier kdgutier deleted the calendar_improvements branch April 6, 2023 17:02
kdgutier added a commit that referenced this pull request Apr 6, 2023
* feat: add alias argument

* fix: mypy hint

* tests

* HINT Unit Test (#499)

* HINT Unit Test
* Changed HINT Unit Test to use percent difference
* Changed default num_samples from 500 to 1000

While this is a good first step, median hierarchical coherence is not guaranteed by the bootstrap reconciliation method.
The unit test needs to be updated in the future for the mean hierarchical coherence.

Obtaining the mean outputs however require additional work on various NF components that we will pospone to the future.

* [FEAT] Allow pandas 2 (#508)

* feat: allow pandas 2

* fix: remove unused cell

* Added MS availability to augment_calendar_df function (#506)

* feat: compute validation loss per epoch (#507)

Co-authored-by: Kin <kin.gtz.olivares@gmail.com>

* solving merge conflict

---------

Co-authored-by: David Luo <60141418+dluuo@users.noreply.github.com>
Co-authored-by: Kin <kin.gtz.olivares@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants