Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updating the notebook after running it at DLMBL2024 #149

Merged
merged 18 commits into from
Sep 6, 2024

Conversation

edyoshikun
Copy link
Contributor

This PR udpates the DLMBL2024 notebook for demo-ing Virtual Staining with VisCy.

After many inputs and debugging from Diane, Anna, and Albert.

Co-authored-by: Diane Adjavon [email protected]
Co-authored-by: AlbertDominguez [email protected]
Co-authored-by: Anna Foix [email protected]

Co-authored-by: Diane Adjavon <[email protected]>
Co-authored-by: AlbertDominguez <[email protected]>
Co-authored-by: Anna Foix <[email protected]>
@mattersoflight
Copy link
Member

@edyoshikun Let's add the solution.ipynb with images embedded.

@ziw-liu ziw-liu added the documentation Improvements or additions to documentation label Aug 30, 2024
@ziw-liu ziw-liu added this to the v0.2.1 milestone Aug 30, 2024
@edyoshikun
Copy link
Contributor Author

the last commit addresses #152

Copy link
Member

@mattersoflight mattersoflight left a comment

Choose a reason for hiding this comment

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

@edyoshikun thanks for iterating on the organization of examples folder. It needs some more cleaning. I suggest:

  • Release 0.2.0
  • Merge this branch into the main after organizing files.
  • Start a small PR that shows a demo of inference for Zernike phase contrast.
  • Release 0.2.1.

@edyoshikun edyoshikun requested a review from ziw-liu September 4, 2024 13:35
pyproject.toml Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
- description on docstrings for inference demo scripts
@edyoshikun edyoshikun dismissed mattersoflight’s stale review September 6, 2024 18:28

the points have been addressed

@edyoshikun edyoshikun merged commit ac437af into main Sep 6, 2024
4 checks passed
@ziw-liu ziw-liu deleted the update_dlmbl2024 branch September 6, 2024 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants