Add validation of transition edges to GS #2454
Closed
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.
Summary:
In a previous diff we decided to add validation that continue_trial_generation flag is the same for all tc that define an edge --> if these are different the gs would have conflicting information how to proceed with generation, and it would be malformed.
Also removing the transition_to optional arg is on the todo list - it requires first reworking the MaxGenerationParallelism criterion, which is why i'm doing this first.
Reviewed By: saitcakmak
Differential Revision: D57248708