numbbo / coco

Numerical Black-Box Optimization Benchmarking Framework
https://numbbo.github.io/coco
Other
260 stars 86 forks source link

C___U_s_e_r_et_al folder name in postprocessing #1783

Closed brockho closed 6 years ago

brockho commented 6 years ago

Since quite some time now, I see that the automatically created folder name in the COCO postprocessing is C___U_s_e_r_et_al on my Windows machine: image

A while ago, this folder name was showing the algorithms (actually algorithm name abbreviations) that were used in the comparison. That was much more helpful...

islandblue commented 6 years ago

This part seems relevant.

https://github.com/numbbo/coco/blob/c83c49b628253a0d708e1bb1f8f625853963ffd3/code-postprocessing/cocopp/findfiles.py#L136-L144

@brockho can you share what command/code you run to get this?

nikohansen commented 6 years ago

The offending line is probably a few lines above

if isinstance(args, str):

which should better read

if not isinstance(args, (list, set, tuple)):

to get around any Python 2/3/unicode problems.

nikohansen commented 6 years ago

May be fixed with 344ad20 in the development branch, @brockho could you test this?

brockho commented 6 years ago

For the record, it was a simple algorithm, I postprocessed to get the error. Something like

import cocopp
cocopp.main('NEWUOA!')

With the latest fix in the development branch, the error is gone. Thanks @nikohansen!