Skip to content

Ds/care flywing notebook#48

Open
diyasrivas wants to merge 2 commits intomainfrom
ds/care_flywing_notebook
Open

Ds/care flywing notebook#48
diyasrivas wants to merge 2 commits intomainfrom
ds/care_flywing_notebook

Conversation

@diyasrivas
Copy link
Member

resolves #47

@diyasrivas diyasrivas self-assigned this Oct 1, 2025
@jdeschamps jdeschamps self-requested a review October 2, 2025 11:50
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.

  • "fly wing ." has an extra space before the period.
  • In the cells with the import, there is a warning for some package. Can you try to update as it says to avoid recording that message?
  • In the data cell, you have local paths to the data. Can you make the cell download the data? You can use something akin to this notebook.
  • I don't understand the need for this part since the data is fixed:
if X_images.ndim == 3:
    axes = "ZYX"
    patch_size = (8, 64, 64)
else: 
    axes = "SZYX" 
    patch_size = (8, 64, 64)
  • Same during predictions

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.

CARE Flywing notebook

2 participants