VEuPathDB / web-eda

Web browser code for EDA-based applications
Apache License 2.0
0 stars 0 forks source link

Issue 1674 add `totalVisibleWithOverlayEntityCount` return value to `useMapMarkers` #1675

Closed bobular closed 1 year ago

bobular commented 1 year ago

Fixes #1674

Also tidied up some comments and merged some reduce functions.

I haven't tested the new totalVisibleWithOverlayEntityCount value. Please could you do that as part of your PR review @adnauseum? It should always be the same as the sum of the marker-center numbers in the viewport.

bobular commented 1 year ago

Cool - best to check it with a variable that isn't available for all entities. "Hookworm, by microscopy" is a good one.

Had no idea about console.table() !!

bobular commented 1 year ago

I've merged it anyway. Thanks for testing!