Hi Jendrik, with the volume of stuff coming in every day at the moment, I don't
want to be the bottleneck for this. If you could find another reviewer for this,
this would be great.
I have implemented this feature here:
https://bitbucket.org/jendrikseipp/downward/pull-requests/37
Can you please have a look at the diff?
If you agree with the overall design, there are at least two open questions:
- Should users be able set time and memory limits for the validator?
- How should we avoid the code duplication for finding the domain file given a
problem file?
One idea from issue569 is to let the driver validate the found plans. The most
important question is probably whether we want to turn on plan validation by
default. I believe we should validate all plans by default and handle VAL just
like the other components with its own options (i.e., add --validate and --
validate-options), but without time and memory limits.