Skip to content

Bioc resub - #85

Merged
sgosline merged 17 commits into
mainfrom
bioc-resub
May 11, 2026
Merged

sgosline merged 17 commits into
mainfrom
bioc-resub

Conversation

@sgosline

Copy link
Copy Markdown
Member

Added the required feature requested by #82.

sgosline and others added 17 commits March 10, 2026 14:34
…_diff_ex.R` such differential expression results and SPE gene list are in the same order for merging. Also clarified comment in code explaining why we don't sort.
…og transformation only happens for non count based data
…nt analysis after fixes to calc_spatial_diff_ex.R
Imputation value writing was handled row by row in the proteomics data, which was incompatible with how Sample Minimum imputation data is calculated. Fixed imputed value assignment method for SampMin to fix that problem.

@ymahlich ymahlich left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added comments / have questions regarding two files. Nothing major just clarification.
Everything else looks good as far as I can tell.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am unclear on the ##herr er hry yhr comment in line 5 but otherwise this looks good

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hah probably a typo

Comment thread R/zzz.R Resolved
@sgosline
sgosline merged commit f50c6ec into main May 11, 2026
1 check failed
@sgosline sgosline mentioned this pull request May 20, 2026
6 of 7 tasks
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