Throwing RestartNeeded from onUpdateParameters() is broken at task creation time about dune HOT 4 CLOSED

Sorry for the insistence, but this report is almost a month old.

I've proposed a fix here on pull-request #81, did any of you guys have some time to take a look?

from dune.

I agree that a first fix would be to at least print an error message, as simply restarting the task likely wont do anything, as the same parameters are read from the same file.

On the other hand, I guess it could be argued that an invalid setting, resulting in a task not being able to start in a known state, should result in DUNE not starting at all.

An alternative would be to start the task with default parameters, but that could also result in some unexpected behaviour, from the users perspective.

I like the fix you proposed in #81.

from dune.

@mariacosta please rule this. I'd say to decide or just open a poll to the community. Otherwise this will be here in perpetuaty.

from dune.

Merged pull request #81.

from dune.