-
Notifications
You must be signed in to change notification settings - Fork 7
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
Update ReadMe and Change Log #73
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
I remember it was mentioned that we should highlight the one change in behaviour that users might notice: labels are not now considered as separate images. I guess this maybe falls into docs rather than changelog (along with the new bfoptions documentation)? |
Yeah, that is a good point. I have added some additional description for that particular setting to note the change in behaviour |
Am I right in thinking that the way ZarrReader looks for Looking at that page, I don't see that either of these methods is mentioned for specifying bfoptions? |
Yeah, that is a good point, I will update the readme to include some details on the bfoptions file usage |
I have updated the ReadMe to remove the outdated TODO section and add documentation for the new reader options. I have also added a link to the release on artifactory
The change log has also been updated to included missing entries from previous releases
Fixes #72