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

Dev #28

Merged
merged 16 commits into from
Apr 15, 2024
Merged

Dev #28

merged 16 commits into from
Apr 15, 2024

Conversation

derrik-gratz
Copy link
Collaborator

generalize PCA, add some new normalization methods for heatmaps

@derrik-gratz
Copy link
Collaborator Author

@micahpf @gktharp1 drawing your attention to this as it changes how the heatmap function works a little. It now just expects a matrix of input values (not necessarily a DESeq object) so this is more generalized for other datatypes. It also pulls the baseline subtraction component out of the heatmap function intonormalizeDataForHeatmap, which has the similar functionality the heatmap used to (provide a grouping variable and a baseline level). So you normalize the data outside of the heatmap (if you so choose), then pass it into the heatmap. There is also normalizeDataForHeatmapByIndividual which is good for timeseries data, allowing you to subtract an individual's baseline from it's other samples.

Example of normalized by individual I kept the baseline samples in to be demonstrative, but they can be omitted from the return of normalizeDataForHeatmapByIndividual.

image

See the previous output here, where it could only be normalized by group. This is still possible using just the normalizeDataForHeatmap function

image

@derrik-gratz
Copy link
Collaborator Author

You don't have to review this (you can if you'd like). I'll push to main at the end of the week

@derrik-gratz derrik-gratz merged commit 5c6cb49 into main Apr 15, 2024
1 check passed
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.

1 participant