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
ISubstanceDescriptor API #137
Conversation
- updated the package - added missing class JavaDoc - fixed method JavaDoc that still pointed to IAtomContainer
Pull most in other request - add the lit ref one now. Note you can overwrite a pullrequest by pushing to the same branch. |
The patches were actually different... I did not overwrite the branch, because I have no clue what would happen to all the comments... and I did not want to loose those. |
John, the other patches had to be applied... I cleaned up stuff there... :( Can you please revert those and apply these? |
So is it just this pull request substancedescriptor3 that needs to be applied I can ignore the other two? |
Okay from what I can tell the first two are still okay? |
I think I changed at least one of them... should I check? |
OK, yes, confirmed: the first two patches are identical. John, you were right! |
Okay fixed now. |
Thanks! |
Please find here the pull request around further substance development. This PR is about the ISubstanceDescriptor API with one example descriptor. If the code is in order, I will backport more descriptors from NanoJava.