-
Notifications
You must be signed in to change notification settings - Fork 14
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
Replace copy of VAL in the jenkins directory #6
Comments
This isn't easy to do, since the state of VAL is still unsettled and its use in SHOP requires the addition of the |
Is it using any VAL-specific functionality? Otherwise you could consider https://github.com/guicho271828/ArriVAL for validation (only for ADL/PDDL though) |
oops I forgot that it was still missing action costs etc. (please forget about it for now) |
I am using an extension to I'm not sure how to get that patch incorporated in the upstream VAL: instead of modifying it slowly, they smashed a huge change in all at once, which makes my patch effectively un-rebase-able. Hence I have to use the old version. (Editorial complaint: this kind of thing would be a lot easier if validate used getopt instead of a hand-written loop) |
I would love to incorporate ArriVAL: VAL's parser is ... not nice. It may have started from a grammar, but by the time the authors were done turning that grammar into something that flex and bison could handle, it was an unreadable mess. |
Replace with another git submodule. Since VAL has bitrotted, it should probably be the patched version that @guicho271828 has.
The text was updated successfully, but these errors were encountered: