TFC: Implement apply panel for structured output #1647
Merged
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.
Closes #1637
Follow-up:
UX Examples
Single panel
Both panels
2023-12-05.14.58.20.mp4
Implementation Notes
While it may look simple and not as helpful without all the deeply structured data in the form of attributes etc. I'm afraid this is as much as we can squeeze out of the structured logs without downloading the whole plan file and rendering all the things with the full knowledge of types etc. This would be a whole different story in terms of complexity though.
I found a few bugs which affected the existing plan log, so while implementing apply log correctly I also fixed them there
listRuns
when that methods is never actually called in thetry
part, it was just an earlier copy-paste error