OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

feat: customize the tab list displayed for different agents

Open Shimada666 opened this issue 1 year ago • 4 comments

Currently, all agents display the same tabs, and some tab functionalities are yet to be implemented, which can confuse users and increase their learning curve. Therefore, I have customized the tab pages for commonly used agents, while other agents still display all tab pages。

MonologueAgent: [FileEditor, Browser], CodeActAgent: [FileEditor, IPython], others: AllTabs

Shimada666 avatar May 03 '24 04:05 Shimada666

image image image

Shimada666 avatar May 03 '24 04:05 Shimada666

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@3886c51). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1543   +/-   ##
=======================================
  Coverage        ?   60.74%           
=======================================
  Files           ?       87           
  Lines           ?     3709           
  Branches        ?        0           
=======================================
  Hits            ?     2253           
  Misses          ?     1456           
  Partials        ?        0           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar May 03 '24 04:05 codecov-commenter

@Shimada666 thanks for doing this!

For supported tabs, I think it should be somewhat inverted:

  • Jupyter tab is only supported by CodeActAgent
  • Plan tab is only supported by PlannerAgent

Browser and Files tabs should be supported by all.

Ideally we'd mark this as metadata on the agent itself, so we don't have to hard-code it into the frontend

rbren avatar May 03 '24 19:05 rbren

@rbren Thank you for your reply! I have already modified my code. Your approach is reasonable, and in the future, I also hope to move this information to the backend as part of the agent metadata, that's a better approach.

Shimada666 avatar May 04 '24 01:05 Shimada666

Looks great! Thanks for this

rbren avatar May 06 '24 00:05 rbren