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

Scatter charts should be on all data, not current page only #211

Open
alexgarel opened this issue Jul 4, 2024 · 0 comments
Open

Scatter charts should be on all data, not current page only #211

alexgarel opened this issue Jul 4, 2024 · 0 comments

Comments

@alexgarel
Copy link
Member

Problem

#189 introduced scatter plot, but they are only based on current page results, while we would like to see results from whole dataset

Proposed solution

In build_charts, gather data we need to get, and relaunch a search with same query but getting only needed data to draw the chart.

Based on the count from the search (as we already launched search):

  • either add a hard cut on the number of results (should be configurable, defaults to 1000)
  • either remove the sort option (little optimization) when we are under the hard cut
@alexgarel alexgarel added the ✨ enhancement New feature or request label Jul 4, 2024
@github-project-automation github-project-automation bot moved this to Backlog (ready for dev) in 🔎 Search-a-licious Jul 4, 2024
@alexgarel alexgarel moved this from Backlog (ready for dev) to Todo (selected) in 🔎 Search-a-licious Jul 4, 2024
@teolemon teolemon removed the ✨ enhancement New feature or request label Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo (selected)
Development

No branches or pull requests

2 participants