-
Notifications
You must be signed in to change notification settings - Fork 893
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
Update last time index code to fix Koalas error #1202
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1202 +/- ##
=======================================
Coverage 98.60% 98.60%
=======================================
Files 130 130
Lines 13932 13932
=======================================
Hits 13738 13738
Misses 194 194
Continue to review full report at Codecov.
|
docs/source/release_notes.rst
Outdated
@@ -5,6 +5,7 @@ Release Notes | |||
**Future Release** | |||
* Enhancements | |||
* Fixes | |||
* Update ``EntitySet.add_last_time_indexes`` to work with Koalas 1.3.0 (:pr:`1202`) |
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.
Can we remove the "Restrict koalas version" entry in the release notes and and add its PR number to this entry?
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.
Done!
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.
Looks good
Update add last time index code to fix Koalas error
When Koalas 1.3.0 was released a new error occurred when setting last time indexes as a result. This PR implements changes to fix this error. The updated code was tested with Koalas 1.1.0, 1.2.0 and 1.3.0 and works fine for each. As a result, the upper version restriction on Koalas in
koalas-requirements.txt
was also removed.