search

MLopez-Ibanez / irace

Iterated Racing for Automatic Algorithm Configuration
https://mlopez-ibanez.github.io/irace/
GNU General Public License v2.0
58 stars 14 forks source link

Change the acotsp executable location for ACOTSP example #23

Closed DE0CH closed 2 years ago

DE0CH commented 2 years ago

Right now the example asks the user to copy the acotsp example to the ~/bin as the target-runner looks for the executable there. This is not ideal because not everything is contained in a folder, and it is not as portable. I hope we can change the target-runner and the documentation so that it runs acotsp directly from the example folder.

MLopez-Ibanez commented 2 years ago

Hi, thanks for your contribution! I don't mind changing this but target-runner does not to run in the same folder as it is found, so "../" may point to a different location.

Maybe change the comment (and quote the strings) to say:

# Path to the ACOTSP executable (this path is relative to the execution directory specified in the scenario).
# EXE="~/bin/acotsp"
EXE="../ACOTSP-1.03/acotsp"

What do you think? If you agree, please rebase and squash commits so I can merge your change without creating a merge commit.

DE0CH commented 2 years ago

Hi, thanks for the review and the suggestion. I just changed the target-runner. In this case ../ACOTSP-1.03/acotsp is relative to the ./acotsp-arena specified in scenario.txt.