Modify

Opened 17 years ago

Closed 16 years ago

#2752 closed defect (fixed)

[patch] WMSPlugin memory usage

Reported by: Upliner Owned by: team
Priority: major Milestone:
Component: Core imagery Version:
Keywords: memory Cc:

Description

WMSPlugin consumes very large amount of memory when zooming in and runs very slow because of swapping. I have found that GeorefImage class creates in memory resized copy of image(reImg) which is very large when zooming in. Maximum size of reImg is 10000x10000 which consumes 300MB of memory each! I propose to lower default maximum resolution to 1000x1000 and allow users to set custom size through wmsplugin.reimg_res property.

Attachments (2)

wmsmem.diff (1.1 KB ) - added by Upliner 17 years ago.
Patch to lower WMSPlugin memory usage
wms_memory_usage.patch (2.9 KB ) - added by xeen 16 years ago.
I mostly agree, however I'm strongly against making this customizeable. For one, users should not have to customize this because because it's dependent on the new/old drawing method and has nothing to do with their memory available. And for two, it will likely cause issues if accidentally set to a wrong value. I've increased the limit to 2000 pixels because otherwise I can have up to 6 images visible which is still slow with the traditional drawing method. I guess 2000 makes a good compromise between this, and each image takes only about 12 MB. Patch removes some old code, too.

Download all attachments as: .zip

Change History (4)

by Upliner, 17 years ago

Attachment: wmsmem.diff added

Patch to lower WMSPlugin memory usage

by xeen, 16 years ago

Attachment: wms_memory_usage.patch added

I mostly agree, however I'm strongly against making this customizeable. For one, users should not have to customize this because because it's dependent on the new/old drawing method and has nothing to do with their memory available. And for two, it will likely cause issues if accidentally set to a wrong value. I've increased the limit to 2000 pixels because otherwise I can have up to 6 images visible which is still slow with the traditional drawing method. I guess 2000 makes a good compromise between this, and each image takes only about 12 MB. Patch removes some old code, too.

comment:1 by xeen, 16 years ago

Summary: WMSPlugin memory usage[patch] WMSPlugin memory usage

comment:2 by stoecker, 16 years ago

Resolution: fixed
Status: newclosed

In [o17522].

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.