New ir #38
Merged
New ir #38
Conversation
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
stage overflow
w ci ground truth
deduplicate in relation btree clean code using same string hash function in c++ and py delete extra output fix docker w
tmp try remove seletction in agg w
interpretor aggregation break interpreter aggregation break ww try fix agg col selection fix ci debuging
w remove dirty file from other branch when 0-arity in head, always 1 count generated?
ww fix a history code add an example w
try fix merge testcase fix truth w w add DeALS to compare
remove dead code
add more example
try fix
Aggregators specify the number of their input and output columns. They no longer specify the arity of the aggregated relation. Aggregation is handled similar to joins now. stage change
fix w w
kmicinski
reviewed
Sep 26, 2023
runslog
Outdated
| @@ -128,7 +128,7 @@ def ensure_backend_built(compile_backend, tsv_bin_path, backend_build_dir, backe | |||
|
|
|||
| def run_local(file, out_path, factloc, cores, verbose, slog_root=None, compile_backend=False, | |||
| debug_flag=False, profile_flag=False, repl_flag=False, compile_only=False, | |||
| overwrite_out=False): | |||
| overwrite_out=False, brouhaha_flag=False): | |||
There was a problem hiding this comment.
I am skeptical that we really should have brouhaha_flag in our scripts on master. I have had bad experiences adding extra infrastructure to research code to support projects like brouhaha--probably we do not want this flag here in master. It is fine to add it in a brouhaha-specific branch, of course.
w satge stage stage resolve all bb Co-authored-by: Kristopher Micinski <[email protected]> stage stage stage stage stage test ex_simple fix switch w stage stage stage w fix
w add v3
fix
limit dump fix slightly better clean and fix pure number string in fact fix ci fix string agian...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.