Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upget.exact.mass(): NullPointerException #73
Comments
rajarshi
referenced this issue
Sep 16, 2018
Merged
Fix documentation and make test compatible with CDK v2.2 #72
This comment has been minimized.
This comment has been minimized.
@sneumann can you confirm this? If same for you, I think this is a historic documentation error as @meowcat handled this differently in RMassBank.
|
This comment has been minimized.
This comment has been minimized.
with latest rcdk this order dependency is gone. if I run the "broken" code from above:
then the functions that failed above now return correct values.
@sneumann can you confirm? Seems that would solve this issue for now? |
This comment has been minimized.
This comment has been minimized.
For the record from John: MolecularFormulaManipulator AtomContainerManipulator |
This comment has been minimized.
This comment has been minimized.
Adding bits & pieces from email conversations here for the record while we look into this: Natural Mass are from BODR - https://github.com/cdk/cdk-build-util/blob/master/src/main/resources/net/sf/cdk/tools/bodr/chemicalElements.xml and think about: |
This comment has been minimized.
This comment has been minimized.
Further bits:
|
sneumann commentedJul 26, 2018
Hi, I can confirm @schymane problem with rcdk-3.4.9 (see MassBank/RMassBank#199). I haven't checked in detail yet, but running the example from the
get.exact.mass
manpage does not work:So either an issue with the rcdk code, my environment or the documentation.
Yours,
Steffen
and