-
Notifications
You must be signed in to change notification settings - Fork 15
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
New features #58
Merged
Merged
New features #58
Changes from 21 commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
baeb799
add to_df, from_df for data records
chjuncn eabbb11
Fix some includes
mikecafarella d173f3c
fix imports in pytest
chjuncn a0db287
fix imports for test
chjuncn 964924f
centralize hash functions into a helper lib
chjuncn 36e4764
Add as_df, from_df methods for DataRecord, so users can easily use th…
chjuncn 15f82b4
Merge branch 'main' of github.com:mitdbg/palimpzest
mikecafarella 53628b4
Merge branch 'main' into new-features
chjuncn 41cfde0
New features (#57)
chjuncn c212b8d
I think we may still want to finish early for LimitScans; will check …
mdr223 6edcd79
Dataset accept list, df as inputs.
chjuncn 670cdcb
Merge branch 'new-features' of https://github.com/mitdbg/palimpzest i…
chjuncn e3b7da2
Merge branch 'main' into new-features
chjuncn 2d80618
resolve merge conflicts
chjuncn 83e5922
Update sets.py
chjuncn dd415f4
demo doesn't write to disk
chjuncn 50dea7b
add pz update --name for pz config
chjuncn 6ba6e36
The changes move away from a single class handling all responsibiliti…
chjuncn e2ae35b
resolved merge conflicts with main; still an error in test_optimizer.…
mdr223 c6dddef
Add option to use plan.run() to execute the plan (Dataset).
chjuncn 1fe61a2
improve the readability of get_champion_model()
chjuncn d1939bb
resolving merge conflicts
mdr223 1ec6746
leave TODO for gerardo
mdr223 5a45ca7
add field types when possible for schemas derived from dataframes
mdr223 a275a07
use for record field access
mdr223 0caa080
map both json str methods --> to_json_str; keep indentation consisten…
mdr223 5548c22
map all as_* verb fcns --> to_* for consistency
mdr223 bb0d3cd
lint check for List --> list, Tuple --> tuple; tried addressing TODO …
mdr223 8fb5af9
assert first physical operator is DataSourcePhysicalOp & use source_o…
mdr223 b0cee01
remove typo
mdr223 95b8ce9
add newline at end of file
mdr223 67751e3
mostly lint changes
mdr223 f194bb2
rename optimize --> get_optimal_plan to clarify purpose a bit
mdr223 2612154
adding note about None for optimization strategy
mdr223 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
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.
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.
Leaving as a mental note that we need to update the other demos as well to use the new execution syntax.
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.
Thank you so much! I'll update demos to adopt the latest execution format.