-
Notifications
You must be signed in to change notification settings - Fork 53
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
Add a method to model cards to generate an overview of sections #302
Comments
@BenjaminBossan I can take this if it's not already taken! |
@lazarust Yes, feel free to give it a shot. Use this thread to discuss designs. |
@BenjaminBossan Here are some of my ideas. I'm thinking this should live in
I'm still thinking through how to dynamically generate this, but what're your thoughts? |
I was thinking about having something like the snippet I posted as a (probably) private method on the We could also add a function that gives a more visually pleasing output based on the private method, like you showed above. I think we can just put it into
We're probably not the first Python lib that tries to visualize a tree in this style. Perhaps there is some code out there that could be used for inspiration. |
@BenjaminBossan After thinking/working on this a little, I wonder if adding the separate function to create the visually appealing version is worth it since users could just use
To me, that seems pretty readable in the terminal, what're your thoughts? Would adding the tree structure be useful? |
@lazarust I'll reply on the actual PR |
This would be nice to have.
Right now, when working with model cards, it's not so easy to get a good overview of what sections currently do or don't exist. The
repr
of the object gives some indication, but besides sections, it also displays the contents, which is distracting. It also truncates long strings, which can obscure the section names.Having a nice overview of the section names, especially when it comes to nested subsections, would be really helpful. Especially when I want to add a subsection to an existing subsection, since I need to know the full name to the subsection, it requires some digging right now.
For now, I'm using something like this:
With that function, I can do something like this:
The text was updated successfully, but these errors were encountered: