search

oar-team / batsim

Batsim: Infrastructure simulator for job and I/O scheduling
GNU Lesser General Public License v3.0
30 stars 15 forks source link

'-' should be allowed in sweep values #42

Closed adfaure closed 6 years ago

adfaure commented 6 years ago 
ERROR Invalid sweep variable workloads: the name got from dict {'filename': '/home/afaure/Projects
/rejectionix/experiments/01_rejection/run/workloads/extracted_CEA-Curie_60H_80util+-0.2_0.json', 'name': 
'CEA-Curie', 'platform': '/home/afaure/Projects/rejectionix/experiments/01_rejection/run/platforms
/platform_5544.xml'} (name=CEA-Curie, got either from the 'name' field if it exists or the first value 
otherwise) is not a valid identifier. It must be because it is used to create files.
mpoquet commented 6 years ago

Yes, this constraint should be removed.
It has been introduced before instance_id. The name was used to create yaml files for each instance.

mpoquet commented 6 years ago

But the way to go is probably not to maintain these two scripts but to create better and more modular ones.