Starting MUSIC with too many MPI process generates cryptic message

[jakobj]

When running a music script in which x executables are defined with x + 1 MPI processes, the following error message results:

music: ../../src/application_mapper.cc:187: void MUSIC::ApplicationMapper::mapApplications(): Assertion `config_->getDict().size() != 0' failed.

This should be replaced with a more informative error message, as it is the case when not enough MPI processes are passed.

Error in MUSIC library: configuration file specifies 3 MPI processes but MUSIC was given 2

I this a genuine bug or just me misinterpeting the output?
I haven't looked at the code yet, so I can't promise a PR.

[mdjurfeldt]

This is a bug and should be trivial to fix. Thank you for the report!

…

On Wed, Oct 18, 2017 at 4:25 PM, Jakob Jordan ***@***.***> wrote: When running a music script in which x executables are defined with x + 1 MPI processes, the following error message results: music: ../../src/application_mapper.cc:187: void MUSIC::ApplicationMapper::mapApplications(): Assertion `config_->getDict().size() != 0' failed. This should be replaced with a more informative error message, as it is the case when not enough MPI processes are passed. Error in MUSIC library: configuration file specifies 3 MPI processes but MUSIC was given 2 I this a genuine bug or just me misinterpeting the output? I haven't looked at the code yet, so I can't promise a PR. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#43>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADcfCWW77BiKNP1AeHYQAg0Y7LjMqzv7ks5stgpXgaJpZM4P9zLc> .

[jakobj]

OK, I'll look into it.