Task #3780 (closed)
BUG: Save/Copy Settings application freeze (Maybe due to the cold weather...)
| Reported by: | jburel | Owned by: | jburel |
|---|---|---|---|
| Priority: | blocker | Milestone: | OMERO-Beta4.3 |
| Component: | General | Version: | n.a. |
| Keywords: | n.a. | Cc: | jamoore, cxallan, jason |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | 0.0d |
| Sprint: | 2011-02-24 (6) |
Description
Tested on nightshade, username: jason
- Select an image
- Change settings e.g. color of a channel
- Click Save.
- Click Copy Settings.
- Force Quit requires to kill the application
Tested on another server, same workflow
It takes a bit of time but the eventually the application is responding.
Change History (12)
comment:1 Changed 13 years ago by jburel
- Component changed from Insight to General
comment:2 Changed 13 years ago by jburel
- Sprint set to 2011-01-27 (24)
comment:3 Changed 13 years ago by jburel
- Summary changed from Save/Copy Settings application freeze (Maybe due to the cold weather...) to BUG: Save/Copy Settings application freeze (Maybe due to the cold weather...)
comment:4 Changed 13 years ago by jburel
- Remaining Time set to 1
comment:5 Changed 13 years ago by jburel
- Sprint changed from 2011-01-27 (4) to 2011-02-10 (5)
When working on rendering engine for big images, will check what's going. Moving to next sprint
comment:6 Changed 13 years ago by jburel
- Sprint changed from 2011-02-10 (5) to 2011-02-24 (6)
Moving to next sprint, priority given to big images
comment:7 Changed 13 years ago by jburel
- Status changed from new to accepted
comment:8 Changed 13 years ago by jburel
Basic test:
Look at simple jpeg, save current settings few times:
Time: 61ms Time: 380ms Time: 409ms Time: 414ms Time: 408ms Time: 412ms
comment:9 Changed 13 years ago by jburel
forgot to indicate that it was using the saveCurrentSettings method
comment:10 Changed 13 years ago by jburel
Problem is coming from the method IPixels#saveCurrentSettings(RenderingDef)
Adding tests.
comment:11 Changed 13 years ago by jburel <j.burel@…>
- Remaining Time changed from 1 to 0
- Resolution set to fixed
- Status changed from accepted to closed
(In [77485b5169807327497a014b2317c6f0c60da204/ome.git] on branch develop) Modify implementation of the saveCurrentSettings method, add tests (close #3780)
comment:12 Changed 13 years ago by jburel <j.burel@…>
(In [573f2db3376f90accd818da88f6c369c34a396f2/ome.git] on branch dev_4_2) Modify implementation of the saveCurrentSettings method, add tests (close #3780)
It seems that the server takes a very long time when few "save" settings calls are made (required one or two more when running locally). Need to review the save method in rendering engine.
Client using the same approach as insight will have the same problem if call not done asynchronously (not the case for the copy action)