-
Notifications
You must be signed in to change notification settings - Fork 1
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
Added the notation *(*) to select all atoms except H in restraints #162
base: main
Are you sure you want to change the base?
Conversation
Build failures are due to a dependency not downloading - hopefully temporary problem. |
Build and test is now OK. Is the 'do not merge' label related to this, or something else? |
Code changes look fine. |
Something else. I need more tests. |
It looks like the new code doesn't warn on non-existent atoms anymore - the failing diff is in lexical.tst output - the old code prints a warning about C(128). The line looks strange though - what's the meaning of the next atom C(2,128)? Comparing files lexical.out and MIN64GHGUI.ORG/LEXICAL.OUT |
I fixed the warning on non-existing atom. |
The piece of work left is this I can work out most of the cases except this one:
The current syntax analyser would probably interpret this as a valid multiplication so will keep the current behaviour. The rest is:
I wondering if 2. and 3. is already possible. I have not checked yet. |
… it anyway (current behaviour)
This reverts commit 5cdfdd3.
rebased branch with master |
No description provided.