Closed VincenzoFerme closed 8 years ago
@VincenzoFerme I think the issue here is not the script. Notice how in our sut plugins repo the folder for wfms is in all lowercaps? https://github.com/benchflow/sut-plugins
in the benchmark configuration files, the field for SUT type, how is wfms written? Because if it's WfMS, meaning with some capital letters, then the scheduler won't find the right configuration because it expects wfms all lowercase, as taken from the sut plugins repo.
@Cerfoglg in the configuration file capitalisation doesn't matter. One could write all possible lowercase/uppercase combinations of the letters w,f,m,s. This is because the correct way to shorten "Workflow Management Systems" is WfMS in the literature :)
It's better if you don't assume lowercase or uppercase when you read field values from test configuration files. For sanity, when you read a field, convert it to all lowercase first (should be just a method call), and then compare its value.
@VincenzoFerme @simonedavico Was worried that capitalisation could have been important at some point, but either way I applied a fix to that https://github.com/benchflow/data-analyses-scheduler/pull/83
An execution log reporting the error as in the title. Moreover in case of an error, the computation must not be marked as succeeded as it seems to be marked from the following logs.
Please test also all the other transformers again in the current state of the system. And add test cases for all the encountered problems.