Add support for locally exported trajectories #2
Conversation
This file contains hidden or 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
- Add OpenHands trajectory format converter - Add upload functionality for local trajectory files - Support viewing OpenHands trajectories in timeline view - Add back button to return to repository view
- Add unit tests for OpenHands trajectory converter - Add unit tests for trajectory upload component - Update documentation with OpenHands support details - Add project structure and usage instructions
neubig
changed the title
- Add specific handling for message types in TimelineStep component - Update openhands-converter to set correct type for messages - Add test to verify message labeling
- Add markdown rendering for message content - Fix type issues in TimelineStep and getStepInfo - Update tests to verify message display
- Update test to match new message display behavior - Move mockGetStepInfo to __mocks__ directory to avoid Jest treating it as a test file
csmith49
reviewed
Mar 7, 2025
|
|
||
| // Add any tool metadata | ||
| if (event.tool_call_metadata) { | ||
| entry.metadata = { |
There was a problem hiding this comment.
Is it worth dumping the rest of the tool call metadata in here as well? There's some other useful info in there (model info, token counts) that might be nice to have available.
There was a problem hiding this comment.
@OpenHands please address this comment
There was a problem hiding this comment.
I opened a new issue #4
csmith49
approved these changes
Mar 7, 2025
|
@OpenHands please address the comment by csmith49 above suggesting that we visualize more information. |
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.
This PR adds support for visualizing trajectories locally exported from OpenHands.
Changes:
Screen.Recording.2025-03-07.at.10.44.56.AM.mov