-
Notifications
You must be signed in to change notification settings - Fork 80
get_anomalies_confidence_interval -> EDA #182
Conversation
model: | ||
model for confidence interval estimation | ||
interval_width: | ||
width of the confidence interval |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's not very clear what it is. Perhaps better would be "The significance level for the confidence interval. The default interval_width = .95 take a 95% confidence interval."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's add at the beginning of sarimax.py
import warnings
from statsmodels.tools.sm_exceptions import ValueWarning
warnings.filterwarnings(
message="No frequency information was provided, so inferred frequency .* will be used",
action='ignore',
category=ValueWarning,
module='statsmodels.tsa.base.tsa_model'
)
And fix Artem's feedback.
Codecov Report
@@ Coverage Diff @@
## master #182 +/- ##
==========================================
+ Coverage 87.97% 88.01% +0.03%
==========================================
Files 75 76 +1
Lines 3477 3505 +28
==========================================
+ Hits 3059 3085 +26
- Misses 418 420 +2
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
IMPORTANT: Please do not create a Pull Request without creating an issue first.
Before submitting (must do checklist)
Type of Change
Proposed Changes
Add get_anomalies_confidence_interval to EDA
Related Issue
Closing issues