-
Notifications
You must be signed in to change notification settings - Fork 1
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
Use custom resolver for query and eval with nested frames. #150
Open
gitosaurus
wants to merge
8
commits into
main
Choose a base branch
from
query-extension
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Commits on Oct 9, 2024
-
Use custom resolver for query and eval with nested frames.
Verify preflighting of nested expressions using AST visitation. Remove logic for splitting queries by string. Now the evaluation is handled by a nested column resolver, and the mixed-mode expressions are preflighted by examining the parsed abstract syntax tree for the query expression.
Configuration menu - View commit details
-
Copy full SHA for 850bd03 - Browse repository at this point
Copy the full SHA 850bd03View commit details -
Import type hint syntax from the future.
Also remove `_ensure_spacing`, no longer used.
Configuration menu - View commit details
-
Copy full SHA for ee57673 - Browse repository at this point
Copy the full SHA ee57673View commit details -
Configuration menu - View commit details
-
Copy full SHA for 838ae68 - Browse repository at this point
Copy the full SHA 838ae68View commit details -
Remove the multi-dimensional error fallback from NestedFrame.query.
This logic was copied from pd.DataFrame.query, and the accompanying comment said that it was to handle an occasional case where `self.loc[b]` would raise an error on a multi-dimensional `b`, but `self[b]` would succeed. I can't cause this error with `.loc` anymore, so the code coverage complains about the unexercised exception clause. Removing it since the limitation that inspired it seems to be gone now.
Configuration menu - View commit details
-
Copy full SHA for 9ad4313 - Browse repository at this point
Copy the full SHA 9ad4313View commit details -
Configuration menu - View commit details
-
Copy full SHA for bc83087 - Browse repository at this point
Copy the full SHA bc83087View commit details
Commits on Oct 10, 2024
-
Rename
NestedSeries
to_SeriesFromNest
.Prevent users from depending on the public interface, and also from assuming that a `NestedSeries` has a nest within it, since that is the existing meaning of that prefix.
Configuration menu - View commit details
-
Copy full SHA for d996fd7 - Browse repository at this point
Copy the full SHA d996fd7View commit details -
Configuration menu - View commit details
-
Copy full SHA for bdce042 - Browse repository at this point
Copy the full SHA bdce042View commit details
Commits on Oct 11, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 67c49ff - Browse repository at this point
Copy the full SHA 67c49ffView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.