Skip to content

Plot values stratified by sequence context#58

Merged
csoneson merged 5 commits intodevelfrom
plotbias
Jan 5, 2026
Merged

Plot values stratified by sequence context#58
csoneson merged 5 commits intodevelfrom
plotbias

Conversation

@csoneson
Copy link
Collaborator

Just a suggestion for a convenience function

@csoneson csoneson requested a review from mbstadler December 31, 2025 10:55
Copy link
Collaborator

@mbstadler mbstadler left a comment

Choose a reason for hiding this comment

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

This looks great, very nice idea!

Just one thought: At the moment, rows are always averaged. I guess that means there is no way to plot the values for individual reads at the moment (unless reads are non-overlapping). Could there be a use-case where we would want to keep the values without averaging per position?

Otherwise feel free to merge!

@csoneson
Copy link
Collaborator Author

Thanks! Yes, fair point - perhaps it would be useful to plot also the individual values, I'll look into that. The other thing I was wondering was whether one would like to be able to facet by sample in the function (at the moment one would have to do that by subsetting the SE, creating a plot for each sample and then combining the plots) 🤔

@mbstadler
Copy link
Collaborator

Yes, I think faceting or coloring by sample might also be a very useful feature, e.g. to directly compare the biases.

@csoneson
Copy link
Collaborator Author

csoneson commented Jan 1, 2026

Ok, I've rewritten the function a bit to make the summarization optional and allow facetting by sample. I also limited the application to read-level assays (since for the summary assays we can't necessarily tell the difference between complete inaccessibility and no coverage).

@mbstadler
Copy link
Collaborator

Looks great! You are very productive already on the first day of the year :-)

@csoneson csoneson merged commit 2a408dd into devel Jan 5, 2026
6 of 8 checks passed
@csoneson csoneson deleted the plotbias branch January 5, 2026 16:11
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