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

New Box Plot ignore "Series Limit" when selecting "Distribute Across" #11669

Closed
2 tasks
eugeniamz opened this issue Nov 12, 2020 · 5 comments
Closed
2 tasks
Assignees
Labels
#bug:cosmetic Cosmetic/layout/design tweak needed !deprecated-label:bug Deprecated label - Use #bug instead viz:charts:boxplot Related to the Boxplot chart

Comments

@eugeniamz
Copy link

Testing new echart Box Plot I notice that when selecting the Distribute Across, it ignores the "Series Limit"

Expected results

If I select "Series Limit", I should always that number of records

Actual results

When I select Distribute Across, Series limit is ignored and shows all the data points

Screenshots

2020-11-12_10-54-21 (1)

How to reproduce the bug

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Environment

Latest Master

Checklist

Make sure to follow these steps before submitting your issue - thank you!

  • I have checked the superset logs for python stacktraces and included it here as text if there are any.
  • [ X] 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.

Additional context

Add any other context about the problem here.

@eugeniamz eugeniamz added the !deprecated-label:bug Deprecated label - Use #bug instead label Nov 12, 2020
@villebro
Copy link
Member

Thanks @eugeniamz , I know what the problem is and there's a few solutions, but I need to digest it a little more to find the optimal one.

@junlincc
Copy link
Member

junlincc commented Nov 12, 2020

Screen Shot 2020-11-12 at 11 04 11 AM

Screen Shot 2020-11-12 at 11 06 26 AM

@villebro also, can we show all x axis labels by default, like the old chart does?

@junlincc junlincc added the #bug:cosmetic Cosmetic/layout/design tweak needed label Nov 12, 2020
@villebro
Copy link
Member

@junlincc
Copy link
Member

@junlincc: apache-superset/superset-ui#834

👍
let's keep rotate 45 by default for now🙏 we can explore 'smarter' solutions post 1.0.

@junlincc junlincc added viz:charts:boxplot Related to the Boxplot chart and removed chart labels Jan 2, 2021
@rusackas
Copy link
Member

Since this thread has gone cold, and revolves around old/unsupported versions of Superset, I'm going to close this issue. If the issue persists, let's by all means open a new issue with current-era screenshots and reproduction steps. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#bug:cosmetic Cosmetic/layout/design tweak needed !deprecated-label:bug Deprecated label - Use #bug instead viz:charts:boxplot Related to the Boxplot chart
Projects
None yet
Development

No branches or pull requests

4 participants