Skip to content

Conversation

@bistline
Copy link
Contributor

@bistline bistline commented Jan 6, 2026

BACKGROUND & CHANGES

This update removes the performance-based restriction of queries less than 50 genes for precomputed dotplot data. This limit remains in effect on the server to handle instances where users can get around the query limit (such as selecting the Heatmap tab when more than 50 genes are already loaded). Small updates to error handling were added to the Heatmap component to handle this, and the error message has been updated to inform users that only precomputed dotplots may load more than 50 genes.

Future enhancements: it would be nice if we could add a visual indication somewhere that some studies/clusters have preprocessed data available. Additionally, for the DE UX, if the data is preprocessed, we can render all genes, not just the 50 currently shown in the table.

MANUAL TESTING

  1. Boot as normal and load any preprocessed study
  2. Open the Network tab in DevTools and find the explore event
  3. Under uniqueGenes, copy out the first 100 gene names (you'll likely have to dump this into a text editor to process into a single line of gene names, space-delimited)
  4. Paste these genes into the gene search and submit
  5. Confirm you get a dot plot with 100 genes
  6. Click the Heatmap tab, and confirm you see the following error:
Screenshot 2026-01-06 at 1 31 53 PM

@bistline bistline requested a review from eweitz January 6, 2026 18:32
@codecov
Copy link

codecov bot commented Jan 6, 2026

Codecov Report

❌ Patch coverage is 27.27273% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.54%. Comparing base (e735add) to head (9023b3a).

Files with missing lines Patch % Lines
...pp/javascript/components/visualization/Heatmap.jsx 0.00% 7 Missing ⚠️
...p/javascript/components/explore/StudyGeneField.jsx 50.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff               @@
##           development    #2342      +/-   ##
===============================================
- Coverage        72.57%   72.54%   -0.03%     
===============================================
  Files              335      335              
  Lines            27919    27924       +5     
  Branches          2587     2588       +1     
===============================================
- Hits             20262    20258       -4     
- Misses            7513     7522       +9     
  Partials           144      144              
Files with missing lines Coverage Δ
...vascript/components/explore/ExploreDisplayTabs.jsx 64.81% <100.00%> (+0.10%) ⬆️
app/lib/study_search_service.rb 85.49% <100.00%> (ø)
...p/javascript/components/explore/StudyGeneField.jsx 58.54% <50.00%> (ø)
...pp/javascript/components/visualization/Heatmap.jsx 3.03% <0.00%> (-0.42%) ⬇️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants