Skip to content

Conversation

@lauraporta
Copy link
Member

Description

What is this PR

  • Bug fix
  • Addition of a new feature
  • Other

Why is this PR needed?
Blogpost on derotation was required!

What does this PR do?
Adds the blogpost with images (linked from the docs). It's basically a summary of the readme and the user guide (key concepts).

References

https://derotation.neuroinformatics.dev/index.html

How has this PR been tested?

Built locally with sphinx-build

Is this a breaking change?

no

Does this PR require an update to the documentation?

no

Checklist:

  • The code has been tested locally
  • Tests have been added to cover all new functionality
  • The documentation has been updated to reflect any changes
  • The code has been formatted with pre-commit

@lauraporta
Copy link
Member Author

Linkcheck fails due to broken links 😓

@JoeZiminski there are some datashuttle links broken, I am not sure to which pages to point them to... are they in the how to guides?

@lauraporta lauraporta marked this pull request as draft July 1, 2025 15:09
@JoeZiminski
Copy link
Member

Hey @lauraporta thanks for flagging this, this was from a recent refactor to the documentation. I opened #164 to fix this problem

@lauraporta lauraporta force-pushed the derotation-blogpost branch from 0d0a7e8 to 2d9557e Compare July 3, 2025 10:54
@lauraporta lauraporta marked this pull request as ready for review July 3, 2025 12:35
@adamltyson adamltyson changed the title Add blogpost Add derotation blogpost Jul 3, 2025
@niksirbi niksirbi self-requested a review July 3, 2025 13:01
Copy link
Member

@niksirbi niksirbi left a comment

Choose a reason for hiding this comment

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

Well done @lauraporta. This is well written and it's satisfying to see how far this project has come along.

I left a few optional suggestions for you to consider.

@lauraporta
Copy link
Member Author

@niksirbi thanks for the suggestions, I've made some changes including a paragraph on the scientific context behind the choice of rotating mice :)

@lauraporta
Copy link
Member Author

Linkcheck fails but the link works 😭

Copy link
Member

@JoeZiminski JoeZiminski left a comment

Choose a reason for hiding this comment

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

Hey @lauraporta I approved with some small suggestions, looks great! It is very clear to read and the images / GIFS are really cool and bring it to life

@lauraporta
Copy link
Member Author

I've applied the suggested modifications! Thank you all

@lauraporta lauraporta force-pushed the derotation-blogpost branch from 3fb4af5 to 6847ce2 Compare July 24, 2025 15:01
@lauraporta lauraporta merged commit 416f159 into main Jul 24, 2025
3 checks passed
@adamltyson adamltyson deleted the derotation-blogpost branch October 24, 2025 12:56
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.

5 participants