broadinstitute / genetic-prevalence-estimator

https://genie.broadinstitute.org/
BSD 3-Clause "New" or "Revised" License
1 stars 0 forks source link

Bug in prevalence/carrier charts when uncheck "include subcontinental populations" #225

Closed sambaxter closed 2 months ago

sambaxter commented 2 months ago

Anne found a bug in v2 lists. When you unselect "include subcontinental populations" the carrier frequency and prevalence drop. The chart stays the same, its just the graph that changes. Below are some screenshots of the example.

https://genie.broadinstitute.org/variant-lists/a603a3f5-9f27-4ff2-ad3d-3dbd1f6abf61/

Screenshot 2024-05-02 at 9 42 13 AM Screenshot 2024-05-02 at 9 43 13 AM
rileyhgrant commented 2 months ago

From a brief glance at the behavior, and as a note to myself for later:

The frequencies that appear to be used to render the bars when the box is unchecked are those of the genetic sub ancestry groups of the East Asian genetic ancestry group. Those also appear to be the two stored prior in the list.

This is likely a matter of double checking the right indices from the array are used to render both the table and the graph.