This PR provides the following refactoring improvements:
System is now a Maven project; dependencies are obtained automatically.
Package args4j have been moved to a Maven dependency. Source code for it has been removed.
Added new options -translatorPath to specify where the Python translator script is located.
Planner is now able to be called from anywhere, not just its root folder (using new options -translatorPath)
Script ./paladinus provided to run the planner, can be put in the PATH and used from anywhere.
Fixed bugs with parsing command line arguments; no errors now when -h is used.
README file reworked significantly.
NOTE: because source code for args4j has been removed and replaced by Maven dependency, we lost the ability to group arguments in "groups" and report the groups in the help. This was carried from myND, where they modified the @Options annotation manually, because annotations cannot be extended as classes, despite being interfaces! See this. The result of this is that all options are printed with -hand without any grouping.
In any case, if any work will be done here, it may be worth investigating other argument parsers , as args4j seems not very maintained and is quite cumbersome I would say...
This PR provides the following refactoring improvements:
args4j
have been moved to a Maven dependency. Source code for it has been removed.-translatorPath
to specify where the Python translator script is located.-translatorPath
)./paladinus
provided to run the planner, can be put in thePATH
and used from anywhere.-h
is used.NOTE: because source code for
args4j
has been removed and replaced by Maven dependency, we lost the ability to group arguments in "groups" and report the groups in the help. This was carried frommyND
, where they modified the@Options
annotation manually, because annotations cannot be extended as classes, despite being interfaces! See this. The result of this is that all options are printed with-h
and without any grouping.In any case, if any work will be done here, it may be worth investigating other argument parsers , as args4j seems not very maintained and is quite cumbersome I would say...
Hope it helps!