Remove pinned pydata version

tl3119 commented 1 year ago

Describe your changes

Issue number

Closes #X

Checklist before requesting a review

[x] Performed a self-review of my code
[x] Formatted my code with pkgmt format
[x] Added tests (when necessary).
[x] Added docstring documentation and update the changelog (when needed)

:books: Documentation preview :books:: https://sklearn-evaluation--335.org.readthedocs.build/en/335/

edublancas commented 1 year ago

@tl3119: this is blocked by https://github.com/ploomber/sklearn-evaluation/issues/336 - let's focus on the other repositories until that one is fixed

edublancas commented 1 year ago

@tl3119 I think this has been fixed. please push an empty commit to trigger the CI, and let me know if all tests pass

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 5339309130

0 of 0 changed or added relevant lines in 0 files are covered.
12 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.004%) to 91.727%

Files with Coverage Reduction	New Missed Lines	%
src/sklearn_evaluation/plot/plot.py	3	73.91%
src/sklearn_evaluation/plot/confusion_matrix_interactive.py	9	93.01%
<!--	Total:	12	-->

Totals
Change from base Build 4674524329:	0.004%
Covered Lines:	3559
Relevant Lines:	3880

Pull Request Test Coverage Report for Build 5339354577

0 of 0 changed or added relevant lines in 0 files are covered.
9 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 91.723%

Files with Coverage Reduction	New Missed Lines	%
src/sklearn_evaluation/plot/confusion_matrix_interactive.py	9	93.01%
<!--	Total:	9		-->

Totals
Change from base Build 4674524329:	0.0%
Covered Lines:	3557
Relevant Lines:	3878

💛 - Coveralls

tl3119 commented 1 year ago

@tl3119 I think this has been fixed. please push an empty commit to trigger the CI, and let me know if all tests pass

readthedocs test fails now

edublancas commented 1 year ago

did you merge from master? https://github.com/ploomber/sklearn-evaluation/pull/337 fixed that

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 5346664763

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.723%

Totals
Change from base Build 5338746980:	0.0%
Covered Lines:	3557
Relevant Lines:	3878

💛 - Coveralls

tl3119 commented 1 year ago

did you merge from master? #337 fixed that

All tests pass now

coveralls commented 7 months ago

Pull Request Test Coverage Report for Build 5339309130

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0 changed or added relevant lines in 0 files are covered.
12 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-8.3%) to 91.727%

Files with Coverage Reduction	New Missed Lines	%
src/sklearn_evaluation/plot/plot.py	3	73.91%
src/sklearn_evaluation/plot/confusion_matrix_interactive.py	9	93.01%
<!--	Total:	12	-->

Totals
Change from base Build 4674524329:	-8.3%
Covered Lines:	3559
Relevant Lines:	3880

ploomber / sklearn-evaluation