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

[BUG] Histogram display not able to handle smaller datasets #373

Closed
Don-Isdale opened this issue May 16, 2024 · 2 comments
Closed

[BUG] Histogram display not able to handle smaller datasets #373

Don-Isdale opened this issue May 16, 2024 · 2 comments

Comments

@Don-Isdale
Copy link
Collaborator

Don-Isdale commented May 16, 2024

Description

Configure display of histograms to handle smaller VCF Genotype datasets such as the pulses Raw data.
Initially display histogram or feature tracks when the axis is opened.
Currently data is not displayed until the user zooms in, and they zoom right out the data display is disabled.

Current Display, before zooming in - datasets are not displayed :
Screenshot from 2024-05-16 14-14-27


Test case

Load Chickpea Raw or Filled-In data on pulses. Open the axes without zooming in : data should display. Also confirm that zooming in and out, both with mouse-wheel, brush/Zoom and Reset, displays data as expected.

Expected Test Result :

Screenshot from 2024-05-13 15-54-57

@Don-Isdale Don-Isdale self-assigned this May 16, 2024
Don-Isdale added a commit that referenced this issue May 17, 2024
…sets

change block field reference .featuresCount to .featureCount in comments : axis-brush.js, api-server.js and mapview.js, and in code : models/block.js : showPaths().

axis-brush.js : brushedBlocks() : add comment that this is distinct from the above, here .featuresCount is correct, although comment : possibly rename this ... to align with block.featureCount. Also comment in axis-brush.hbs.

services/data/block.js : getSummary() :  set .featureCount from featuresCountsResultSum().
viewedChartable() : tag log message with featuresCounts because that is a key dependency.
ember-serializer.js : update the import ENV to pretzel-frontend/config/environment, previously ../../config/environment.
@kennyAgricultureVic
Copy link
Contributor

Tested and verified that the changes work
image

@kennyAgricultureVic kennyAgricultureVic changed the title Display of histograms to handle smaller datasets [BUG] Histogram display not able to handle smaller datasets May 23, 2024
@kennyAgricultureVic
Copy link
Contributor

Deployed on https://pulses.plantinformatics.io/
Marking ticket as complete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants