Ticket #672 (closed assessed: fixed)

Opened 9 years ago

Last modified 9 years ago

Model does not save after crash

Reported by: shermanl Owned by:
Priority: normal Milestone: Ecopath 6: release 6.0.8
Component: Database / Import Version:
Severity: major Keywords:
Cc:

Description

This seems to be a database issue. I'm guessing it has to do with the model being in an unsafe state. It does not save any thing in ecosim (after doing a fit to timeseries).

To replicate:
1) Open Model
2) Change a Fishery>Landing
3) Save (it saves ok)
4) Open Fit to TS (using TEEB scenario)
5) Set Sensitivities (in anyway)
6) Run a search
7) You'll see vulnerabilities change.
8) Save the model and you won't see V's change. In fact the model crashes.

Attachments

FAO 61.zip (1.2 MB) - added by shermanl 9 years ago.

Change History

Changed 9 years ago by shermanl

comment:1 Changed 9 years ago by jeroens

  • Priority changed from NOT SET to normal
  • Type changed from NOT SET to assessed
  • Severity changed from block to major

comment:2 Changed 9 years ago by jeroens

Oh wow, the code somehow seems to forget that Ecosim is loaded! On it.

comment:3 Changed 9 years ago by jeroens

  • Status changed from new to closed
  • Resolution set to fixed

This issue was an artifact caused by the way feedback messages were handled, and crept into the code over last weekend. Fixed.

comment:4 Changed 9 years ago by shermanl

Found an easier way to reproduce this. Using any TEEB database models:

1) Open model
2) Open Scenario (TEEB)
3) Load Time Series (TEEB)
4) Change Vulnerabilities
5) Save - does not allow to save.

This process does not fail if you skip step 3.

comment:5 Changed 9 years ago by shermanl

  • Status changed from closed to reopened
  • Resolution fixed deleted

comment:6 Changed 9 years ago by jeroens

  • Status changed from reopened to closed
  • Resolution set to fixed

Fixed a bug in the save time series logic. Any attempt to save an ecosim model with TS loaded failed on this.

Note: See TracTickets for help on using tickets.