Unpin dependencies from [project.optional-dependencies]#4431
Unpin dependencies from [project.optional-dependencies]#4431MichaelFu512 merged 20 commits intomainfrom
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4431 +/- ##
=======================================
+ Coverage 99.7% 99.7% +0.1%
=======================================
Files 352 352
Lines 40035 40041 +6
=======================================
+ Hits 39914 39920 +6
Misses 121 121 ☔ View full report in Codecov by Sentry. |
| "astroid <= 2.6.6", | ||
| "Sphinx >= 5.0.0", | ||
| "nbconvert >= 6.5.0", | ||
| "nbsphinx >= 0.8.5, < 0.9.0", |
There was a problem hiding this comment.
A combination of nbsphinx, astroid, and docutils cause the readthedocs pipeline to fail. I'll need to open a separate ticket to uncap those.
MichaelFu512
requested review from
christopherbunn,
eccabay,
jeremyliweishih and
thehomebrewnerd
| assert automl.results["pipeline_results"][0]["mean_cv_score"] == 1.0 | ||
|
|
||
| with pytest.raises(AttributeError, match="set attribute"): | ||
| with pytest.raises(AttributeError): |
There was a problem hiding this comment.
Do you know why the changes lines in this file and in test_pipeline_search_plots.py are showing as uncovered by tests? That seems a bit strange to me.
thehomebrewnerd
left a comment
thehomebrewnerd
left a comment
There was a problem hiding this comment.
Seems fine overall, but I'm a bit concerned/confused about the changed lines that show up as uncovered by codecov.
4 participants
Pull Request Description
Closes #4430
After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of
docs/source/release_notes.rstto include this pull request by adding :pr:123.