search

overture-stack / maestro

Indexing software to connect SONG and Arranger through Elasticsearch
https://www.overture.bio/documentation/maestro/
GNU Affero General Public License v3.0
0 stars 0 forks source link

Age range stats edge case shows wrong Min and Max #238

Closed justincorrigible closed 2 years ago

justincorrigible commented 2 years ago

Steps to reproduce (while in Dev with mock data):

  1. move min slider to 31 (range should be [31-40])
  2. select female, which sets the max to 30.

Expected result: The range input shows the min === max as 30.

Actual result: The range input shows the min as 20, even though all docs are Age 30.

image.pngimage.png

Note: After studying the origin of the stats, it seems to be a computed value from either GraphQL or ES.

justincorrigible commented 2 years ago

After further analysis, turns out the min is lower due to the two unselected docs (male), which include other ages. The issue is in the way the min is displayed as overlapping the max, which makes it seem unusable/disabled.

To be improved via UX. Closing ticket. image.png