NVlabs / FPSci

Aim Training Experiments
Other
70 stars 23 forks source link

FPSci crashes if no `startupconfig.Any` is present #277

Closed jspjutNV closed 3 years ago

jspjutNV commented 3 years ago

You can reproduce this by downloading v21.02.01 and double clicking FirstPersonScience.exe. On first run, it will create the startupconfig.Any and crash. The second run after the file is created works as expected.

bboudaoud-nv commented 3 years ago

This was introduced by #262, in that the default StartupConfig constructor does not populate the experimentList with defaultExperiment if no experiments are present unless loading from Any.

Rather than simply adding this manually, I'm debating adding a StartupConfig::load() static method that will handle this and always use the Any constructor rather than needing to keep 2 different constructors in sync.