-
Notifications
You must be signed in to change notification settings - Fork 3
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
Customized unit #46
Comments
I just add one thing here. The same code works on Windows. So there is something about the "set_attributes()" function from the EML 2 package doesn't work the same between Mac and PC. |
Thank you Li for the report. Can you report the error message on Macs? If
you suspect it's because of set_attributes, then we should report to the
EML R folks.
This is potentially a bigger issue as MetaEgress isn't tested on really
anything but my version of Windows yet.
…On Mon, Sep 23, 2019 at 6:16 PM Li Kui ***@***.***> wrote:
I just add one thing here. The same code works on Windows. So there is
something about the "set_attributes()" function from the EML 2 package
doesn't work the same between Mac and PC.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#46?email_source=notifications&email_token=AKAZD5WSSQBVBRPTSBJFMVDQLFE3PA5CNFSM4IZUUFJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7MRP2Y#issuecomment-534321131>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AKAZD5TIJB3ZURP546J53ATQLFE3PANCNFSM4IZUUFJA>
.
|
yes, probably a good idea to report to EML R team. I could do that. |
There was actually an issue couple months ago about this: ropensci/EML#278 |
Thanks for taking care of it. I haven't run into this issue myself, but it
seems people do run into it on Windows, not just Mac.
…On Tue, Sep 24, 2019 at 3:36 PM Li Kui ***@***.***> wrote:
There was actually an issue couple months ago about this: ropensci/EML#278
<ropensci/EML#278>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#46?email_source=notifications&email_token=AKAZD5U26OT2Q7NVNE3RGELQLJ25JA5CNFSM4IZUUFJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7PXDGI#issuecomment-534737305>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AKAZD5UGDV6LTURJKAPGSGLQLJ25JANCNFSM4IZUUFJA>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The current code doesn't work for the customized unit. For example, if you put it an unit "milligramPerGramPerHourPerPhotonFlux", the create_entity function fails.
I run the code on Mac.
The text was updated successfully, but these errors were encountered: