multiscale / muscle3

The third major version of the MUltiScale Coupling Library and Environment
Apache License 2.0
25 stars 13 forks source link

No need for argc/argv any more in the Instance constructor? #200

Open LourensVeen opened 1 year ago

LourensVeen commented 1 year ago

The manager injects the instance name and manager location via environment variables now. So if you always start your models through the launcher, you don't need argc/argv. Maybe make them optional, or remove them? Do we want to continue to support launching the simulation through a launch script or docker-compose or something? And if the latter, can't we just use environment variables there too?