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

Fix column order in checkTargetFiles function #72

Closed ivan1arriola closed 2 weeks ago

ivan1arriola commented 3 weeks ago

This commit updates the checkTargetFiles function to reorder the columns of the configurations, ensuring that the .ID. column is the first column. This change addresses an issue with irace.assert failing due to mismatched column orders when comparing configurations with initial configurations provided by the user.

MLopez-Ibanez commented 3 weeks ago

Thanks, could you add a test for this that will fail without your change? I still do not understand why this is needed.

MLopez-Ibanez commented 2 weeks ago

Thanks for the PR, it is very welcome. Although I ended up committing a slightly different fix (with a testcase), I have mentioned you in the NEWS file: https://mlopez-ibanez.github.io/irace/news/index.html#fixes-development-version

If you have any other fixes or improvements, do not hesitate to submit them!