search

INCF / MUSIC

MUSIC, the MUltiSimulation Coordinator
GNU General Public License v3.0
37 stars 37 forks source link

Starting MUSIC with too many MPI process generates cryptic message #43

Open jakobj opened 6 years ago

jakobj commented 6 years ago

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 commented 6 years ago

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 notifications@github.com wrote:

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

music: ../../src/applicationmapper.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 https://github.com/INCF/MUSIC/issues/43, or mute the thread https://github.com/notifications/unsubscribe-auth/ADcfCWW77BiKNP1AeHYQAg0Y7LjMqzv7ks5stgpXgaJpZM4P9zLc .

jakobj commented 6 years ago

OK, I'll look into it.