Skip to content

Added ChIP-seq Large Margin Visualization to Gallery#51

Open
ANAMASGARD wants to merge 2 commits intoanimint:gh-pagesfrom
ANAMASGARD:fix/gallery-issue-39-chip-seq
Open

Added ChIP-seq Large Margin Visualization to Gallery#51
ANAMASGARD wants to merge 2 commits intoanimint:gh-pagesfrom
ANAMASGARD:fix/gallery-issue-39-chip-seq

Conversation

@ANAMASGARD
Copy link
Copy Markdown
Contributor

ChIP-seq Large Margin Viz (FIXES #39)

Source Code

Deployed Viz

Gallery Update

  • Added ANAMASGARD/chip-seq-large-margin to repos.txt
  • Generated screenshot: repos/ANAMASGARD/chip-seq-large-margin.png

Screenshot from prior gallery build; regenerate with animint2pages after
deploying the updated penalty plot height in chip-seq-large-margin-code.
@ANAMASGARD ANAMASGARD force-pushed the fix/gallery-issue-39-chip-seq branch from e8f8e67 to 4fd608c Compare March 22, 2026 16:44
@ANAMASGARD ANAMASGARD requested a review from tdhock March 22, 2026 16:56
@tdhock
Copy link
Copy Markdown
Contributor

tdhock commented Mar 24, 2026

there are still problems.
too many green lines.
you are missing a showSelected.
also text size is still too small.
can you please double check these, and check that it looks and works the same as the original, before asking for me to review?

@ANAMASGARD
Copy link
Copy Markdown
Contributor Author

Sir @tdhock,
I have updated the code and regenerated the visualization to address all the issues you pointed out.

  • https://anamasgard.github.io/chip-seq-large-margin/

  • Added showSelected="sample.id" to the relevant segments and widerects. Green bars in the errors and peaks panels now only appear for the currently selected sample.

  • Increased size for both the sample name labels and the blue regression labels so they're comfortably readable alongside the axis text.

  • Switched from geom_label_aligned back to geom_text with manually adjusted y-positions. The box borders are gone and the labels match the original layout.

Screenshot From 2026-04-09 08-50-24

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.

chip-seq large margin data viz

2 participants