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

Fix Mu Character. #8

Closed

Conversation

ilan-gold
Copy link

The mu character appeared to somewhat non-functioning previously. I'm not 100% sure what is going on here but the character now prints as μ instead of µ which seems more correct. @LauraKuett could you test this out to confirm?

@ilan-gold ilan-gold closed this Jun 4, 2021
@ilan-gold
Copy link
Author

I am at a bit of a loss here @LauraKuett - it should be possible to get a greek character to print but I'm having trouble. Will keep hammering away.

@ilan-gold
Copy link
Author

@LauraKuett and @jwindhager I opened an issue on image.sc about this: https://forum.image.sc/t/micro-character-in-omexml-from-python/53578/4

@jwindhager
Copy link
Contributor

Hi Ilan, thanks so much for digging and your report on image.sc. Seems like an encoding issue indeed. I remember switching from UTF-8 to ASCII because of #4 reported by our software engineer. But from what @cgohlke wrote on image.sc, the warning about non-ASCII seems to be a bug in tifffile and the OME-XML should be UTF-8. I'll double-check over the weekend and update xtiff as soon as there is a fix available for tifffile (if that's the case).

@jwindhager
Copy link
Contributor

xtiff 0.7.6 now writes OME-XML as UTF-8 again (technically not supported by TIFF, but required by OME-XML/the microns character). tifffile's ASCII encoding (and associated error message - not a bug, apologies for not reading the thread on image.sc properly!) is skipped by passing the encoded byte string to tifffile. Please let me know if that works for you @ilan-gold. Thanks again for digging @ilan-gold and for the tifffile support @cgohlke!

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