Skip to content

Solara based User Interface#1854

Merged
tpike3 merged 4 commits intomesa:mainfrom
ankitk50:ui_layout2
Nov 3, 2023
Merged

Solara based User Interface#1854
tpike3 merged 4 commits intomesa:mainfrom
ankitk50:ui_layout2

Conversation

@ankitk50
Copy link
Copy Markdown
Contributor

Features of the user interface

  • Title bar with name of the example/model.
  • Controls moved to sidebar which can be collapsed to expose bigger space.
  • Draggable and Resizable space elements.
Screenshot 2023-10-31 at 23 49 33

@ankitk50 ankitk50 mentioned this pull request Oct 31, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 1, 2023

Codecov Report

Attention: 41 lines in your changes are missing coverage. Please review.

Comparison is base (6a39efd) 81.38% compared to head (172ccf9) 77.42%.
Report is 17 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1854      +/-   ##
==========================================
- Coverage   81.38%   77.42%   -3.97%     
==========================================
  Files          15       15              
  Lines         881      970      +89     
  Branches      186      213      +27     
==========================================
+ Hits          717      751      +34     
- Misses        141      191      +50     
- Partials       23       28       +5     
Files Coverage Δ
mesa/experimental/jupyter_viz.py 34.61% <41.42%> (-0.06%) ⬇️

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Member

@tpike3 tpike3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ankitk50. I just had a few requested changes and was curious about one aspect of rendering the browser. Then we can get this merged and out the door.

@rht and @jackiekazil

@tpike3
Copy link
Copy Markdown
Member

tpike3 commented Nov 2, 2023

Could you take a look at the build failures as well please

@ankitk50
Copy link
Copy Markdown
Contributor Author

ankitk50 commented Nov 3, 2023

seems like I need to add some tests to improve the code coverage.

Copy link
Copy Markdown
Member

@tpike3 tpike3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tpike3 tpike3 merged commit 4ec7433 into mesa:main Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants