Closed PascalLesage closed 5 years ago
Incomplete. Closed and new pull request coming up.
Reopening pull request with all commits.
Looks good. Was a bit of a shock for me to see the consolidated as the default parameter loader, but I think it makes sense.
The "unconsolidated" IndexedParameterMapping is available in .parameter_data_loaded
PackagesDataLoader.parameters now contains a
ConsolidatedParameterIndexedMapping
object.Fixes https://github.com/PascalLesage/presamples/issues/56#issue-455297560 Fixes https://github.com/PascalLesage/presamples/issues/57#issue-456283943 Helps with https://github.com/PascalLesage/presamples/issues/21#issue-297985764
ConsolidatedParameterIndexedMapping
tested .