search

felipeangelimvieira / prophetverse

A multiverse of Prophet models for timeseries
https://prophetverse.com
Apache License 2.0
33 stars 3 forks source link

[MNT] Remove mark.skip from check_estimator tests #89

Closed felipeangelimvieira closed 3 months ago

felipeangelimvieira commented 4 months ago

Run sktime check estimator tests in CI

Closes #47

codecov[bot] commented 4 months ago

Codecov Report

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

Project coverage is 93.77%. Comparing base (2d41525) to head (c6b8299). Report is 2 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #89 +/- ## ========================================== + Coverage 93.60% 93.77% +0.16% ========================================== Files 26 26 Lines 1188 1188 ========================================== + Hits 1112 1114 +2 + Misses 76 74 -2 ```

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

felipeangelimvieira commented 4 months ago

Possibly memory issues in github actions? May be related to actions/runner-images#6680

felipeffm commented 4 months ago

I tested locally and indeed the test_check_estimator exploded my memory.

felipeangelimvieira commented 3 months ago

Closing in favour of #94