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

Add CellSens and Trestle to the fs-lite lists #364

Closed
wants to merge 1 commit into from

Conversation

melissalinkert
Copy link
Member

See #6693.

@ghost
Copy link

ghost commented Sep 17, 2012

Adding trestle to the fs-lite list results in #9233.

@ghost
Copy link

ghost commented Sep 17, 2012

CellSens is likewise broken with fs-lite.

@chris-allan
Copy link
Member

Based on past discussions with @melissalinkert my understanding was that both are either multi file or directory based formats which are unsupported with fs-lite due to the munging that happens under /OMERO/Files. I'm not even sure these would be detected properly without file extensions?

/cc @rleigh-dundee

@ghost
Copy link

ghost commented Sep 17, 2012

CellSens doesn't appear to work at all. Strangely, trestle does, so I'm not sure exactly what gets transferred server-side, but it looks OK apart from the corruption in some images (which looks like an issue with tile offsets being miscalculated rather than the image being absent per se).

@melissalinkert
Copy link
Member Author

Right, I forgot about the single-file requirement for fs-lite. Trestle is likely only (sort of) working because only one file is strictly necessary - so it should be detected correctly, but something else is going wrong to cause the tiles to be read incorrectly.

This branch has already been pulled out of the 4.4.4 builds, but feel free to just close without merging if desired.

@joshmoore
Copy link
Member

@rleigh-dundee suggests closing, since this will now be completely handled by fs-lite.

@joshmoore joshmoore closed this Sep 26, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants