Modify

Ticket #2752 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

[patch] WMSPlugin memory usage

Reported by: Upliner Owned by: team
Priority: major Component: Plugin wms
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

wmsmem.diff Download (1.1 KB) - added by Upliner 3 years ago.
Patch to lower WMSPlugin memory usage
wms_memory_usage.patch Download (2.9 KB) - added by xeen 2 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.

Change History

Changed 3 years ago by Upliner

Patch to lower WMSPlugin memory usage

Changed 2 years ago by xeen

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 Changed 2 years ago by xeen

  • Summary changed from WMSPlugin memory usage to [patch] WMSPlugin memory usage

comment:2 Changed 2 years ago by stoecker

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

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.