Skip to content
This repository was archived by the owner on Apr 15, 2026. It is now read-only.

Finished ophys viz section#7

Merged
CodyCBakerPhD merged 5 commits intomainfrom
more_polish
Feb 6, 2026
Merged

Finished ophys viz section#7
CodyCBakerPhD merged 5 commits intomainfrom
more_polish

Conversation

@CodyCBakerPhD
Copy link
Copy Markdown
Owner

@CodyCBakerPhD CodyCBakerPhD commented Feb 4, 2026

And finished clipping/rephrasing prompt questions

Also swapped to colon after verbage implying 'look at code below'

Also added custom pre-commit for cleaning certain notebook outputs for simpler diff (and easier conflicts)

Quick link: https://github.com/CodyCBakerPhD/dandi-open-data-tutorial/blob/037a0f40b020f80a9496861400d753d7662ba24e/Get-to-know-a-Dandiset.ipynb

@CodyCBakerPhD CodyCBakerPhD self-assigned this Feb 4, 2026
Copy link
Copy Markdown
Collaborator

@h-mayorquin h-mayorquin left a comment

Choose a reason for hiding this comment

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

Do you want to leave the printed nwbfile?

It would be better if this table was left-aligned like the rest of the HTML things you added:
image

LGTM, feel free to merge.

@CodyCBakerPhD
Copy link
Copy Markdown
Owner Author

It would be better if this table was left-aligned like the rest of the HTML things you added:

I think I left a comment about that on previous PR - in my IDE it is, in GitHub I don't know how to make it that way

Do you?

I will try putting in a designated div with left anchor maybe...

Do you want to leave the printed nwbfile?

The raw printed, yes (also shouldn't cause conflicts since deterministic and not a binary file like an image

The HTML, no - those auto expand everything and look terrible in GitHub as a consequence (wish I could stop it from happening)

@CodyCBakerPhD CodyCBakerPhD merged commit c97f1f3 into main Feb 6, 2026
1 check passed
@CodyCBakerPhD CodyCBakerPhD deleted the more_polish branch February 6, 2026 21:48
@CodyCBakerPhD CodyCBakerPhD mentioned this pull request Feb 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants