Skip to content

Commit a783788

Browse files
andyclaude
andcommitted
Builder Geometry Trace opens on the total energy; only a user's pick is remembered
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BTQ7mniRtDdhU7VF9tEVBD
1 parent e21de82 commit a783788

2 files changed

Lines changed: 11 additions & 2 deletions

File tree

‎src/apps/builder/GeomTracePropertyPanel.C‎

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,7 @@ GeomTracePropertyPanel::GeomTracePropertyPanel()
7272
p_recompute(false),
7373
p_fold(true),
7474
p_currentProp(""),
75+
p_propChosen(false),
7576
p_importantProps(),
7677
p_keyToId(),
7778
p_idToKey()
@@ -89,6 +90,7 @@ GeomTracePropertyPanel::GeomTracePropertyPanel(IPropCalculation *calculation,
8990
p_recompute(false),
9091
p_fold(false),
9192
p_currentProp(""),
93+
p_propChosen(false),
9294
p_importantProps(),
9395
p_keyToId(),
9496
p_idToKey()
@@ -247,7 +249,11 @@ void GeomTracePropertyPanel::restoreSettings()
247249
config->Read("GeomTrace/Delay",&p_delay,500);
248250
p_currentProp = config->Read("GeomTrace/Prop","");
249251
if (p_currentProp.empty() || !getCalculation()->getProperty(p_currentProp)) {
250-
// find first important prop that actually exists to fill plot
252+
// Open on the total energy when there is one; else the first series.
253+
if (getCalculation()->getProperty("TEVEC") && p_keyToId.count("TEVEC")) {
254+
p_currentProp = "TEVEC";
255+
return;
256+
}
251257
map<string,wxWindowID>::iterator string_to_id_it;
252258
for (string_to_id_it = p_keyToId.begin();
253259
string_to_id_it != p_keyToId.end();
@@ -267,7 +273,8 @@ void GeomTracePropertyPanel::saveSettings()
267273
config->Write("GeomTrace/Recompute", p_recompute);
268274
config->Write("GeomTrace/Fold", p_fold);
269275
config->Write("GeomTrace/Delay", p_delay);
270-
config->Write("GeomTrace/Prop", wxString(p_currentProp));
276+
if (p_propChosen)
277+
config->Write("GeomTrace/Prop", wxString(p_currentProp));
271278
}
272279

273280

@@ -341,6 +348,7 @@ void GeomTracePropertyPanel::OnToolClick(wxCommandEvent& event)
341348
p_fold = event.IsChecked();
342349
} else {
343350
p_currentProp = p_idToKey[event.GetId()];
351+
p_propChosen = true;
344352
fillPlot();
345353
}
346354
saveSettings();

‎src/apps/builder/GeomTracePropertyPanel.H‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,7 @@ class GeomTracePropertyPanel : public VizPropertyPanel,
7777
bool p_fold;
7878

7979
string p_currentProp;
80+
bool p_propChosen; // the user picked the series (so it is remembered)
8081

8182
vector<string> p_importantProps;
8283
map<string,wxWindowID> p_keyToId;

0 commit comments

Comments
 (0)