Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

initial UI #241

Closed
wants to merge 9 commits into from
Closed

initial UI #241

wants to merge 9 commits into from

Conversation

XuhuiZhou
Copy link
Member

Closes #
#151

πŸ“‘ Description

This PR provides a streamlit interface/API for SOTOPIA

βœ… Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed
  • Branch name follows type/descript (e.g. feature/add-llm-agents)
  • Ready for code review

β„Ή Additional Information

Copy link

codecov bot commented Oct 29, 2024

Codecov Report

Attention: Patch coverage is 0.33784% with 590 lines in your changes missing coverage. Please review.

Project coverage is 62.78%. Comparing base (61f190e) to head (e8fff7b).

Files with missing lines Patch % Lines
...pia/ui/streamlit_ui/pages/render_chat_websocket.py 0.00% 200 Missing ⚠️
...topia/ui/streamlit_ui/rendering/rendering_utils.py 0.00% 107 Missing ⚠️
sotopia/ui/streamlit_ui/utils.py 0.00% 106 Missing ⚠️
...otopia/ui/streamlit_ui/rendering/render_episode.py 0.00% 73 Missing ⚠️
...otopia/ui/streamlit_ui/pages/display_characters.py 0.00% 40 Missing ⚠️
sotopia/ui/streamlit_ui/app.py 0.00% 30 Missing ⚠️
sotopia/ui/streamlit_ui/pages/display_scenarios.py 0.00% 19 Missing ⚠️
sotopia/ui/streamlit_ui/pages/display_episodes.py 0.00% 5 Missing ⚠️
sotopia/ui/streamlit_ui/pages/intro.py 0.00% 4 Missing ⚠️
sotopia/ui/streamlit_ui/__init__.py 0.00% 3 Missing ⚠️
... and 2 more
@@             Coverage Diff             @@
##             main     #241       +/-   ##
===========================================
- Coverage   74.47%   62.78%   -11.70%     
===========================================
  Files          61       72       +11     
  Lines        3162     3754      +592     
===========================================
+ Hits         2355     2357        +2     
- Misses        807     1397      +590     
Files with missing lines Coverage Ξ”
sotopia/ui/fastapi_server.py 97.95% <66.66%> (-0.99%) ⬇️
sotopia/ui/streamlit_ui/rendering/__init__.py 0.00% <0.00%> (ΓΈ)
sotopia/ui/streamlit_ui/__init__.py 0.00% <0.00%> (ΓΈ)
sotopia/ui/streamlit_ui/pages/intro.py 0.00% <0.00%> (ΓΈ)
sotopia/ui/streamlit_ui/pages/display_episodes.py 0.00% <0.00%> (ΓΈ)
sotopia/ui/streamlit_ui/pages/display_scenarios.py 0.00% <0.00%> (ΓΈ)
sotopia/ui/streamlit_ui/app.py 0.00% <0.00%> (ΓΈ)
...otopia/ui/streamlit_ui/pages/display_characters.py 0.00% <0.00%> (ΓΈ)
...otopia/ui/streamlit_ui/rendering/render_episode.py 0.00% <0.00%> (ΓΈ)
sotopia/ui/streamlit_ui/utils.py 0.00% <0.00%> (ΓΈ)
... and 2 more

@XuhuiZhou XuhuiZhou marked this pull request as draft October 29, 2024 18:04
@XuhuiZhou XuhuiZhou added the ui label Nov 20, 2024
@XuhuiZhou XuhuiZhou requested a review from ProKil December 4, 2024 04:01
@XuhuiZhou XuhuiZhou marked this pull request as ready for review December 4, 2024 04:01
@ProKil ProKil changed the base branch from main to demo December 5, 2024 05:00
@XuhuiZhou XuhuiZhou closed this Dec 7, 2024
@XuhuiZhou XuhuiZhou deleted the feature/sotopia-demo branch December 7, 2024 04:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant