Skip to content

MicroSplit notebook for lightning API#39

Open
CatEek wants to merge 7 commits intomainfrom
iz/feat/ms_lightning_api
Open

MicroSplit notebook for lightning API#39
CatEek wants to merge 7 commits intomainfrom
iz/feat/ms_lightning_api

Conversation

@CatEek
Copy link

@CatEek CatEek commented Sep 16, 2025

Add a notebook draft.

Uses newly proposed configuration pipeline(see careamics #514)

Performance haven't been tested, most likely there're bugs in the config.

Should be viewed after #40

Copy link
Member

Choose a reason for hiding this comment

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

It seems your PR are sharing commits, where they shouldn't!

"# save the image\n",
"im = Image.fromarray(cover * 255)\n",
"im = im.convert(\"L\")\n",
"im.save(\"BSD68_Noise2Void_lightning_api.jpeg\")"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"im.save(\"BSD68_Noise2Void_lightning_api.jpeg\")"
"im.save(\"HT_LIF_MicroSplit_lightning_api.jpeg\")"

Copy link
Member

@jdeschamps jdeschamps left a comment

Choose a reason for hiding this comment

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

This branch needs to be rebased once the HDN example has the comments included.

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