Skip to content

render side with most synapses#172

Merged
leburnett merged 2 commits into
mainfrom
issue_52
May 18, 2026
Merged

render side with most synapses#172
leburnett merged 2 commits into
mainfrom
issue_52

Conversation

@floesche
Copy link
Copy Markdown
Member

@floesche floesche commented May 17, 2026

For neuron types whose contra lateral innervation is larger than the ipsi lateral, it now looks like this:

image

instead of

image

Other neurons will also highlight the side for both, the celltype and ROI, in color.

The ROIs will be show in the correct location in the combined page. This also means, the Left cell will show on the "right side", the right cell on the "left side":

image

Also, the eyemaps for left and right side now have the same size.

This partially addresses #52

@floesche floesche requested a review from leburnett May 17, 2026 16:55
@floesche floesche self-assigned this May 17, 2026
@floesche floesche linked an issue May 17, 2026 that may be closed by this pull request
Base automatically changed from cleanup-dead-code to main May 18, 2026 19:28
Copy link
Copy Markdown
Collaborator

@leburnett leburnett left a comment

Choose a reason for hiding this comment

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

Works well for CT1 and DNc02 and is a good solution. I tried aMe12 because I knew that that was a bilateral cell but, as you mention, this fix is only for cells that predominantly innervate the contralateral side. Capturing the innervation of bilateral cells in both hemispheres is out of the scope of this PR. Merging in now.

@leburnett leburnett merged commit 901475e into main May 18, 2026
8 checks passed
@leburnett leburnett deleted the issue_52 branch May 18, 2026 20:23
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