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

Fixed backtest test. #235

Merged
merged 1 commit into from
Aug 12, 2019
Merged

Fixed backtest test. #235

merged 1 commit into from
Aug 12, 2019

Conversation

jaheba
Copy link
Contributor

@jaheba jaheba commented Aug 12, 2019

The test implied that we wanted to test the output of backtest_metrics. However, we relied on an already checked-in file, which was never changed.

Also, I've removed the __str__ method on DatasetStatistics since we otherwise can't evaluate the output.

@jaheba jaheba force-pushed the fix-backtest-test branch from 4bc17c2 to ee83fef Compare August 12, 2019 23:12
@jaheba jaheba force-pushed the fix-backtest-test branch from ee83fef to 77ae46d Compare August 12, 2019 23:14
@lovvge lovvge assigned lovvge and unassigned lovvge Aug 12, 2019
@szha
Copy link
Member

szha commented Aug 12, 2019

Job PR-235/3 is complete.
Docs are uploaded to http://gluon-ts-staging.s3-accelerate.dualstack.amazonaws.com/PR-235/3/index.html

@codecov-io
Copy link

codecov-io commented Aug 12, 2019

Codecov Report

Merging #235 into master will decrease coverage by 0.06%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #235      +/-   ##
==========================================
- Coverage   81.25%   81.19%   -0.07%     
==========================================
  Files         130      130              
  Lines        7395     7392       -3     
==========================================
- Hits         6009     6002       -7     
- Misses       1386     1390       +4
Impacted Files Coverage Δ
src/gluonts/dataset/stat.py 95.49% <ø> (-1.85%) ⬇️
src/gluonts/evaluation/backtest.py 85.05% <33.33%> (-2.45%) ⬇️

@jaheba jaheba merged commit c93445a into awslabs:master Aug 12, 2019
@jaheba jaheba deleted the fix-backtest-test branch August 12, 2019 23:27
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.

4 participants