-
Notifications
You must be signed in to change notification settings - Fork 428
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
set snapshot as extra params to get graph data of correct version in groot #1666
Conversation
…oid potential cyclice dependencies while using the library in pegasus.
…o tag_name in sink op
[IR Runtime] IR TagOpt for more compact structure in Runtime
[IR Runtime] Support EdgeExpandWithIntersection and Unfold Op in Runtime
Revert "[IR Core] Support Union of Intersection in IR-Core"
…ction Revert "[IR Runtime] Support EdgeExpandWithIntersection and Unfold Op in Runtime"
Codecov Report
@@ Coverage Diff @@
## main #1666 +/- ##
===========================================
- Coverage 73.18% 40.32% -32.86%
===========================================
Files 89 89
Lines 9736 9736
===========================================
- Hits 7125 3926 -3199
- Misses 2611 5810 +3199
Continue to review full report at Codecov.
|
LGTM, a little question: does this PR pass the e2e tests by hand since our CI doesn't have tests for the snapshot? I mean, the correctness of the PR is checked manually, not only compiled and unittested. Or we can add a few tests that focus on the snapshot, if applicable. |
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
What do these changes do?
compiler fetches the snapshot from groot storage and pass to runtime as extra params to get graph data of correct version
Related issue number
Fixes #1566