Confirm that the "seed path" setting in user's config (yaml) file is being honored in all cases

uaf-arctic-eco-modeling / dvm-dos-tem

A process based Dynamic Vegetation, Dynamic Organic Soil, Terrestrial Ecosystem Model.

MIT License

22 stars 24 forks source link

Confirm that the "seed path" setting in user's config (yaml) file is being honored in all cases #667

Closed tobeycarman closed 7 months ago

tobeycarman commented 10 months ago

Make sure that the seed path that the user selects is being honored for seeding parameter values. Need to check this for both the Sensitivity class and the MadsTEMDriver class.

jsclein-uaf commented 9 months ago

YES, I tried changing the seed_path during a 'ca' run and it did not work and YES having this option available during the 'sa' runs would be great and overall for all calibrating so that you could beupdating your parameter files in a separate location, and not have to worry when you are updating a version of TEM

Benjamin-Maglio commented 9 months ago

Tested commit sha 71429f2929c837f96a36c430422b991245b4faee and seed path is honored for CA. Working on honoring for SA now.

jsclein-uaf commented 9 months ago

thanks Ben, looking forward to seed path working in SA run too... thanks for all your help and hard work. joy

tobeycarman commented 7 months ago

fixed with PR #699