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

In text, display model list in markdown table format #118

Open
JasonWeill opened this issue Apr 25, 2023 · 0 comments
Open

In text, display model list in markdown table format #118

JasonWeill opened this issue Apr 25, 2023 · 0 comments
Labels
enhancement New feature or request @jupyter-ai/magics
Milestone

Comments

@JasonWeill
Copy link
Collaborator

Problem

As @dlqqq noted in #100, in the results of %ai list, it may be possible to use libraries like py-markdown-table to generate markdown tables declaratively, based on a data structure, instead of imperatively, as we currently do. Some of our logic is declared twice, once for text and once for markdown format.

Proposed Solution

Use py-markdown-table, or a similar library, to generate markdown for the model list.

Use markdown-style tables for text output.

Remove remaining helper functions that use different logic for text and markdown parts.

@JasonWeill JasonWeill added the enhancement New feature or request label Apr 25, 2023
@JasonWeill JasonWeill added this to the Future milestone Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request @jupyter-ai/magics
Projects
None yet
Development

No branches or pull requests

1 participant