Simplify Markdown table generation in magic commands #118
Assignees
Labels
enhancement
New feature or request
good first issue
Good for newcomers
scope:magics
Issues concerning the magic commands
status:triaged
Issues that have been fully triaged
Milestone
Comments
dlqqq
changed the title
dlqqq
changed the title
|
@keerthi-swarna is working on a fix for this in #1243. Mentioning Keerthi here so I can assign this issue to Keerthi. |
|
Oh, actually I need to add Keerthi to the project before I can assign Keerthi an issue. I'll do that now. Reference: https://stackoverflow.com/questions/41171882/how-to-assign-an-issue-to-non-contributors |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
The text was updated successfully, but these errors were encountered: