-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #2424 output mapping serialization #2425
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -114,6 +114,7 @@ private void exportSimulationToFile(Simulation simulation, string moBiFile) | |
var simulationTransfer = new SimulationTransfer | ||
{ | ||
Simulation = moBiSimulation, | ||
OutputMappings = simulation.OutputMappings, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. saving the actual output mapping when exporting |
||
JournalPath = _journalRetriever.JournalFullPath | ||
}; | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -27,7 +27,7 @@ public SimulationTransfer Load(string pkmlFileFullPath) | |
{ | ||
var project = _projectRetriever.CurrentProject; | ||
|
||
//use new ObjectBaseRepository here as the resulting simulation will be registered later on when added to the project | ||
//use new WithIdRepository here as the resulting simulation will be registered later on when added to the project | ||
var simulationTransfer = _simulationPersister.Load(pkmlFileFullPath, new WithIdRepository()); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Interestingly, we do not load obs data when loading from MoBi (this is not really a use case that is done often). So I am not going to tackle this for now |
||
|
||
project?.Favorites.AddFavorites(simulationTransfer.Favorites); | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -291,7 +291,7 @@ public static SensitivityParameter SensitivityParameter(string name = "Sensitivi | |
|
||
public static DataRepository IndividualSimulationDataRepositoryFor(string simulationName) | ||
{ | ||
var simulationResults = new DataRepository("Results"); | ||
var simulationResults = new DataRepository(); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Id was fixed here. Letting the code create the id automatically |
||
var baseGrid = new BaseGrid("Time", TimeDimensionForSpecs()) | ||
{ | ||
Values = new[] {1.0f, 2.0f, 3.0f} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change related to the change of interface