scikit-learn-contrib / MAPIE

A scikit-learn-compatible module to estimate prediction intervals and control risks based on conformal predictions.
https://mapie.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
1.27k stars 102 forks source link

Add Sequential Predictive Conformal Inference (SPCI) method for Time Series #373

Open Kevin-Chen0 opened 10 months ago

Kevin-Chen0 commented 10 months ago

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes #370

Type of change

Please remove options that are irrelevant.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist

LacombeLouis commented 10 months ago

Hey @Kevin-Chen0, Note that I would fork from the following branch: 334-adaptive-conformal-predictions-for-time-series. Looking forward to see your PR!

Kevin-Chen0 commented 10 months ago

Hey @Kevin-Chen0, Note that I would fork from the following branch: 334-adaptive-conformal-predictions-for-time-series. Looking forward to see your PR!

Hi @LacombeLouis I have just merged the latest scikit-learn-contrib:334-adaptive-conformal-predictions-for-time-series branch into Tempor-ai:370-sequential-predictive-conformal-inference-for-time-series. So SCPI will adhere to the ACI structure. Please lmk if you see anything else. Thx!

codecov-commenter commented 9 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (614293e) 100.00% compared to head (c458e17) 100.00%. Report is 10 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #373 +/- ## ========================================== Coverage 100.00% 100.00% ========================================== Files 39 39 Lines 4616 4735 +119 Branches 487 780 +293 ========================================== + Hits 4616 4735 +119 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Kevin-Chen0 commented 6 months ago

Hi @thibaultcordier, my friend @grgstock will tackle this SPCI PR moving forward. What is the best way for him to reach out to you and your team in case he has specific questions about the existing MAPIE code base?

thibaultcordier commented 6 months ago

Hi @thibaultcordier, my friend @grgstock will tackle this SPCI PR moving forward. What is the best way for him to reach out to you and your team in case he has specific questions about the existing MAPIE code base?

Hi @Kevin-Chen0, @grgstock, the best way for him to contact our team is to contact us directly via email (see the AUTHORS.rst file) and in PR conversation. We will take the time to definitively integrate your PR into the MAPIE code by agreeing on the structure of the code to keep.