-
Notifications
You must be signed in to change notification settings - Fork 349
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
chore: pin pandas and ray versions for ray tests #8932
Conversation
✅ Deploy Preview for determined-ui canceled.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8932 +/- ##
=======================================
Coverage 44.41% 44.41%
=======================================
Files 721 721
Lines 134936 134936
Branches 2237 2235 -2
=======================================
Hits 59929 59929
Misses 74849 74849
Partials 158 158
Flags with carried forward coverage won't be shown. Click here to find out more. |
waiting on this run to finish https://app.circleci.com/pipelines/github/determined-ai/determined?branch=hz-new-ami |
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.
lgtm, thanks for taking care of this
thanks for the help |
Co-authored-by: Ilia Glazkov <ilia.glazkov@hpe.com>
* avoid pinning pandas for other tests --------- Co-authored-by: Ilia Glazkov <ilia.glazkov@hpe.com>
https://github.com/search?q=repo%3Aray-project%2Fray+pandas+language%3AText&type=code&l=Text (cherry picked from commit 6d1a596)
Description
Test Plan
Commentary (optional)
Checklist
docs/release-notes/
.See Release Note for details.
Ticket