This repository was archived by the owner on May 15, 2026. It is now read-only.
Enhanced file viewing in OH-ACI#126
Closed
adityasoni9998 wants to merge 7 commits into
Closed
Conversation
2 tasks
Contributor
|
#129 is merged so I'm gonna close this one. Thank you! |
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
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
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.
Description
Integrated Markdown converters from OpenDeepResearch, which in-turn borrows it from Magentic-One. This allows viewable several file-types like PDFs, audios, PPTs, Spreadsheets, etc. by converting them to Markdown using open-source libraries.
Related Issue
Motivation and Context
How Has This Been Tested?
Run test.py with any file of your choice to test the changes. The following extensions have been tested: ".xlsx", ".pptx", ".wav", ".mp3", ".m4a", ".flac", ".pdf", ".docx", ".csv", ".html", ".htm", ".txt", and all files viewable in plain-text format.
Just take some example files of each of the above types and run test.py to understand how this works.
Does this PR introduce a breaking change?
The only problem is the view-range parameter of the view function in OH-ACI. This was disable for OpenHands-Versa, but you may want to have it in OH-ACI in general. One way would be to modify prompt of str_replace_editor tool in OpenHands that this parameter is only support for text-like files.