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

How is viz.py works #8685

Closed
liuhao-121 opened this issue Nov 28, 2019 · 5 comments
Closed

How is viz.py works #8685

liuhao-121 opened this issue Nov 28, 2019 · 5 comments
Labels
inactive Inactive for >= 30 days

Comments

@liuhao-121
Copy link

I know when i need a new chart , i need modify viz.py to add a new class which will support the view and get data , response;
but i can only find where it defined, find no where and how it works

@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the label #question to this issue, with a confidence of 0.92. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@liuhao-121
Copy link
Author

@vliweihan

@jackyq2015
Copy link

I also want to do the similar thing. All the UI code is under:
https://github.com/apache-superset/superset-ui-plugins

Also in the readme page you can see their recommendation:

@superset-ui/legacy-* packages are extracted from the classic Apache Superset and converted into 
plugins. These packages are extracted with minimal changes (almost as-is). They also depend on 
legacy API (viz.py) to function.


@superset-ui/plugin-* packages are newer and has higher quality in general. A key difference that 
they do not depend on viz.py (which contain visualization-specific python code) and interface with 
/api/v1/query/ instead: a new generic endpoint instead meant to serve all visualizations. Also should 
be written in Typescript.

@vliweihan
Copy link

@liuhao-121 I found some introductions online about how to add a new visualization , It only works for older versions. The file architecture of present version of superset seems very different. If you use the previous version of 0.28 and include 0.28, you can try it.

#3013

@stale
Copy link

stale bot commented Jan 28, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. For admin, please label this issue .pinned to prevent stale bot from closing the issue.

@stale stale bot added the inactive Inactive for >= 30 days label Jan 28, 2020
@stale stale bot closed this as completed Feb 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inactive Inactive for >= 30 days
Projects
None yet
Development

No branches or pull requests

3 participants