Ticket #45 (closed: fixed)
Ecosim and Ecospace scenario dialogues can ask multiple times to load a scenario
Reported by: | joeb | Owned by: | administrator |
---|---|---|---|
Priority: | normal | Milestone: | Ecopath 6: build 6.0.7 |
Component: | Overall GUI | Version: | |
Severity: | minor | Keywords: | |
Cc: |
Description
This can get pretty confusing but basically. Turn off the Save window layout option open a model and go straight to Run ecopsace you will be asked for an Ecosim and Ecospace scenario at least twice. Click around in the interface once you have the Run ecospace interface up you may be asked to reload the scenarios. This also happens when the Save window layout option is on but it seem even more confusing then because things are loading in the background and you dont know why you are being asked to load a scenario.
Change History
comment:2 Changed 16 years ago by jeroens
Debug findings:
- Loading Ecospace will cause PoolColors? to be loaded. (in RunEcospace?.InitColors?())
- This will cause the color manager to load colors and to stick them in the ecopath groups. This will cause Variable validation at Ecopath level, causing the core state monitor to enter EcopathLoaded? state, forgetting that Ecospace was just loaded!
- The Core controller then tries to revive the Ecospace state by showing the load dialogs.
- Ecospace will eventually be loaded. The dialogs have appeared twice.
comment:3 Changed 16 years ago by jeroens
Possible solutions at load dialog level, but this needs to be thought out really well:
- There is no need to physically reload scenarios (either sim or space) when Ecopath has been modified. Once Ecopath has ran, the current sim and space scenarios are still valid and usable.
- Maybe the ecosim and ecospace dialogs need to be invoked explicitly rather than implicitly from nav tree manipulation. That way, a current loaded scenario remains valid until the user actively decides to load a different scenario.
- The Core controller is unable to distinguish whether the user would want to keep using a current sim/space scenario or would like to load a new scenario
However, these changes may have conceptual implications...
Parent of #48