Skip to content
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 #801 no refresh of gof plots #805

Merged
merged 1 commit into from
Nov 17, 2022
Merged

Conversation

msevestre
Copy link
Member

No description provided.


if (simulation.Chart != null)
xElement.Add(chartSerializer.Serialize(simulation.Chart, serializationContext));
addSerializedChart(simulationElement, simulation.Chart, serializationContext);
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The charts were not saved in the simulation and also not serialized

@@ -158,38 +156,44 @@ public override void Edit(object subject)

private void loadChart()
{
if (_simulationPredictedVsObservedChartPresenter.Chart == null)
{
var chart = _chartFactory.Create<SimulationPredictedVsObservedChart>();
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was creating a local chart in the presente.r gone as you as you close it

if (_simulation.PredictedVsObservedChart == null)
_simulation.PredictedVsObservedChart = _chartFactory.Create<SimulationPredictedVsObservedChart>();

if(_simulationPredictedVsObservedChartPresenter.Chart == null)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This code can be refactored for sure but this will be gone as soon as we are done with 11.1 so I don't think it needs optimizing. That being said, I'd like to know if this is right.
What I don't understand is why the identity curven is not drawn the first time (or the residual zeros)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let me see if there's a fast fix for that identity/residual 0 problem today.

@rwmcintosh rwmcintosh merged commit b3d51c0 into develop Nov 17, 2022
@rwmcintosh rwmcintosh deleted the 801-no-refresh-of-gof-plots branch November 17, 2022 12:29
rwmcintosh pushed a commit that referenced this pull request Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants