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

Conformal Prediction #592

Merged
merged 22 commits into from
Aug 1, 2023
Merged

Conformal Prediction #592

merged 22 commits into from
Aug 1, 2023

Conversation

kvnkho
Copy link
Collaborator

@kvnkho kvnkho commented Jul 24, 2023

Some things:

  1. I am least confident about the MSTL implementation if you can take a took.

  2. Am I right in thinking that prediction_intervals=ConformalIntervals(h=13, n_windows=2) means that you need to do forecast(h=13), otherwise the shapes of the intervals will not match and you can't form the output DataFrame. I feel like we should either catch this and raise an explicit error rather than a numpy shape error, or we can handle implicitly and override?

  3. The type hint for level: Optional[Tuple[int]] causes a mypy issue because of the following behavior:

test = (1, 2)
print(type(test))            # tuple
print(type(sorted(test)))    # list

so the line:

levels = sorted(levels)

casts tuple to list, which mypy doesn't like. So I think all the levels hints should be Optional[List[int]]

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@kvnkho kvnkho marked this pull request as draft July 24, 2023 00:57
@kvnkho kvnkho changed the title [DRAFT] Conformal Prediction Conformal Prediction Jul 24, 2023
@kvnkho kvnkho marked this pull request as ready for review July 28, 2023 05:05
@kvnkho kvnkho requested a review from AzulGarza July 28, 2023 05:05
Copy link
Member

@AzulGarza AzulGarza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @kvnkho! This is awesome! 🥳

Could we clean the notebooks to merge the PR?🙌

@kvnkho
Copy link
Collaborator Author

kvnkho commented Aug 1, 2023

Thank you @kvnkho! This is awesome! 🥳

Could we clean the notebooks to merge the PR?🙌

Finished cleaning I think.

@AzulGarza AzulGarza self-requested a review August 1, 2023 17:22
@AzulGarza AzulGarza enabled auto-merge (squash) August 1, 2023 17:22
@AzulGarza AzulGarza merged commit 478e774 into main Aug 1, 2023
16 checks passed
@AzulGarza AzulGarza deleted the feat/conformal_prediction branch August 1, 2023 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants