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

word cloud series limit is in invalid #3882

Closed
3 tasks done
dqzgzjhzjbhzl opened this issue Nov 16, 2017 · 7 comments · Fixed by #3902
Closed
3 tasks done

word cloud series limit is in invalid #3882

dqzgzjhzjbhzl opened this issue Nov 16, 2017 · 7 comments · Fixed by #3902

Comments

@dqzgzjhzjbhzl
Copy link

dqzgzjhzjbhzl commented Nov 16, 2017

Make sure these boxes are checked before submitting your issue - thank you!

  • I have checked the superset logs for python stacktraces and included it here as text if any
  • I have reproduced the issue with at least the latest released version of superset
  • I have checked the issue tracker for the same issue and I haven't found one similar

Superset version

0.20.4

Expected results

the drop-down box named series limit is working

Actual results

the drop-down box seems like doing nothing

Steps to reproduce

just select different value in the drop-down box

@amoussoubaruch
Copy link

@dqzgzjhzjbhzl what did you mean by drop-down box? Which dataviz type did you want to use?

@dqzgzjhzjbhzl
Copy link
Author

dqzgzjhzjbhzl commented Nov 16, 2017

@amoussoubaruch i want to use Word Cloud.
word cloud

@amoussoubaruch
Copy link

@dqzgzjhzjbhzl Change your metric and test

@dqzgzjhzjbhzl
Copy link
Author

@amoussoubaruch in the red box , i select 5, then the number of name in the Word Cloud should be 5. is it right?

@amoussoubaruch
Copy link

I thinks that series limit parameter is like select topN. So i select only 5 row in your dataset

@dqzgzjhzjbhzl
Copy link
Author

@mistercrunch so this is a bug

@mistercrunch
Copy link
Member

a squashed bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants