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

Refactor CYXNgffBaseTest #134

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Refactor CYXNgffBaseTest #134

wants to merge 2 commits into from

Conversation

tischi
Copy link
Collaborator

@tischi tischi commented Jan 27, 2023

No description provided.

@tischi
Copy link
Collaborator Author

tischi commented Jan 27, 2023

Hi @constantinpape

we started adding code to also test higher resolution levels.
We tried with @tibuch data and could not immediately see any issues.
The value was > 0 for channel = 3 and resolution = 2.

It would be nice if you could extend the tests a bit more for your data, to be sure and then we could merge this.

We will look now in mobie-viewer-fiji whether the issue is there.

@constantinpape
Copy link
Collaborator

We probably don't see an issue because the chunks are 1. I will double check this later.

@tischi
Copy link
Collaborator Author

tischi commented Jan 30, 2023

We also tried it with @tibuch's data which has chunking and we also did not find the issue...
But it would be great if you could add test data with channel chunking anyway!

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