display variables in marimo #7967
Conversation
|
Display a single variable ... 
... from a larger model ... 
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7967 +/- ##
==========================================
- Coverage 91.48% 91.48% -0.01%
==========================================
Files 116 116
Lines 18962 18969 +7
==========================================
+ Hits 17348 17353 +5
- Misses 1614 1616 +2
🚀 New features to boost your workflow:
|
ericmjl
left a comment
ericmjl
left a comment
There was a problem hiding this comment.
LGTM! Do you reckon we'll need to start testing the marimo-related features?
| ) | ||
|
|
||
| # https://docs.marimo.io/guides/integrating_with_marimo/displaying_objects/#option-1-implement-a-_display_-method | ||
| rv_out._display_ = types.MethodType(functools.partial(safe_display, model=model), rv_out) |
There was a problem hiding this comment.
I think it's a good time as any to remove pretty print from the variables and leave it to the model graphviz only.
There was an issue/ PR to remove those others for ipython. This is never going to be exhaustive because there are many other ways of creating variables for models
There was a problem hiding this comment.
So does that mean, removing other pretty print methods as well? I'm not following your comment fully.
There was a problem hiding this comment.
Yes, only the model remains with pretty print
|
This goes in the opposite direction I suggest we should take in #7712 Also graph representation instead of textual by default seems very verbose |
3 participants
Description
This uses the
model_to_mermaidfunction for generating graph for a specific variable in the model.Method bound to:
Related Issue
Checklist
Type of change