Skip to content

Comments

Integrate DE notebook#1573

Closed
asizemore wants to merge 16 commits intomainfrom
feature-1567-integrate-de-notebook
Closed

Integrate DE notebook#1573
asizemore wants to merge 16 commits intomainfrom
feature-1567-integrate-de-notebook

Conversation

@asizemore
Copy link
Member

Work in progress.

Currently - replaced fold change search with differential expression as we await the differential expression search to work (awaiting tuning manager).
But we can see how it looks in the search page. Also note i'm using Cristina's dev site with old data so we don't have the right collections yet.

Screen.Recording.2025-12-19.at.2.06.44.PM.mov

@asizemore
Copy link
Member Author

Forgive me - i didn't git add all the files for the previous commits! Here it comes...

@asizemore
Copy link
Member Author

Just added some code (includes temporary workarounds because of data issues) that will allow us to hide the pca configuration component cell in the notebook. That way it can just take the one rnaseq collection it needs and run the compute automatically for the user.

Screen.Recording.2025-12-22.at.12.22.42.PM.mov

Still to do:

  • make sure the collection passes the collectionPredicate
  • the compute notebook cell needs to handle messaging if the compute errs.

This new feature of hiding the compute configuration can only be tested with /eda and mbio data currently. The lee gambien data right now only has wgcna module collections which fail the pca compute.

@bobular
Copy link
Member

bobular commented Feb 3, 2026

Closing in favour of #1590

@bobular bobular closed this Feb 3, 2026
@bobular bobular deleted the feature-1567-integrate-de-notebook branch February 3, 2026 18:06
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.

2 participants