-
Notifications
You must be signed in to change notification settings - Fork 835
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
graph level metadata #1803
graph level metadata #1803
Conversation
Wed May 6 20:10:12 UTC 2020 impatient try |
Wed May 6 20:10:21 UTC 2020 impatient try |
Thu May 7 15:05:04 UTC 2020 impatient try |
Thu May 7 15:05:12 UTC 2020 impatient try |
Tue May 12 16:33:45 UTC 2020 impatient try |
Tue May 12 16:33:45 UTC 2020 impatient try |
Wed May 13 16:37:11 UTC 2020 impatient try |
Wed May 13 16:37:15 UTC 2020 impatient try |
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.
Good stuff!
A few nit picks in the code.
The most important area to me to look at more closely before approving is the situations where nil, nil
is returned from GetShapeFromGraph
function.
Understand there is a design doc here so will take a look at that.
Mon May 18 15:14:37 UTC 2020 impatient try |
Mon May 18 15:14:37 UTC 2020 impatient try |
Tue May 19 11:30:05 UTC 2020 impatient try |
Tue May 19 11:30:08 UTC 2020 impatient try |
Tue May 19 12:05:33 UTC 2020 impatient try |
Tue May 19 12:05:43 UTC 2020 impatient try |
Tue May 19 15:06:20 UTC 2020 impatient try |
Tue May 19 15:06:23 UTC 2020 impatient try |
/test integration |
Tue May 19 19:07:09 UTC 2020 impatient try |
Fri May 22 11:50:59 UTC 2020 impatient try |
Fri May 22 11:51:02 UTC 2020 impatient try |
Fri May 22 11:51:48 UTC 2020 impatient try |
Fri May 22 11:52:04 UTC 2020 impatient try |
Fri May 22 11:53:25 UTC 2020 impatient try |
Fri May 22 11:54:59 UTC 2020 impatient try |
/retest |
/test notebooks |
Fri May 22 13:44:48 UTC 2020 impatient try |
Fri May 22 13:44:52 UTC 2020 impatient try |
Fri May 22 13:45:16 UTC 2020 impatient try |
/retest |
Fri May 22 13:54:28 UTC 2020 impatient try |
The failing notebook is now passing locally without troubles - have not a single clue what is going wrong here. |
/test integration |
Sun May 24 19:07:53 UTC 2020 impatient try |
Sun May 24 19:08:12 UTC 2020 impatient try |
/test integration |
Tue May 26 08:54:36 UTC 2020 impatient try |
Tue May 26 08:54:50 UTC 2020 impatient try |
Tue May 26 08:55:38 UTC 2020 impatient try |
/retest |
Tue May 26 09:28:49 UTC 2020 impatient try |
@RafalSkolasinski: The following tests failed, say
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
@axsaucedo all tests passed locally, shall we merge manually? |
failed to trigger Pull Request pipeline
|
Initial implementation of graph-level metadata. Closes #1728
Implementation around following new structs defined in executor
note that struct
GraphMetadata
defines output format of graph level metadata