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

feat: upgrade to gpt-vis #1852

Closed
wants to merge 5 commits into from
Closed

feat: upgrade to gpt-vis #1852

wants to merge 5 commits into from

Conversation

hustcc
Copy link
Collaborator

@hustcc hustcc commented Aug 20, 2024

Description

Use gpt-vis to render the content returned by the model, and define a markdown output protocol to support various visual charts for presentation.

How Has This Been Tested?

Manually test all markdown output and page rendering by gpt-vis.

Snapshots:

Include snapshots for easier review.

Checklist:

  • My code follows the style guidelines of this project
  • I have already rebased the commits and make the commit message conform to the project standard.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • Any dependent changes have been merged and published in downstream modules

GPT-VIS

  • MarkdownVis -> GPTVis.
  • .less file.
  • type define error.
  • rehypePlugins props etc.
  • defaultRenderer for code.

@github-actions github-actions bot added the enhancement New feature or request label Aug 20, 2024
csunny
csunny previously approved these changes Aug 21, 2024
Copy link
Collaborator

@csunny csunny left a comment

Choose a reason for hiding this comment

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

LGTM

@hustcc
Copy link
Collaborator Author

hustcc commented Aug 22, 2024

还需要测试,暂时不要合并。

@hustcc hustcc closed this Aug 23, 2024
@hustcc hustcc deleted the upgrade-gpt-vis branch August 23, 2024 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants