Ticket #1830 (closed feature: fixed)

Opened 22 months ago

Last modified 18 months ago

Give the bbox strategy an option to make it more agressive

Reported by: tschaub Owned by: crschmidt
Priority: minor Milestone: 2.8 Release
Component: Strategy Version: 2.7
Keywords: Cc:
State: Complete

Description

The bbox strategy currently only decides that the existing feature set is invalid if the existing bounds don't contain the new bounds. There are cases where it makes sense to be more aggressive with regard to requesting new data. Several of these cases are addressed by considering the change in resolution between requests in addition to the bounds. This still belongs in a bbox strategy because it is about managing successive requests for data within some bounding box. It is sort of like saying "request new data if the user navigates beyond the old bounds or is viewing an area much smaller/bigger than the previous request". One way to capture the smaller/bigger area than previous request is to add a resolution factor.

The resFactor option works in both ways (area getting bigger or smaller). If you set a resFactor of 2 and you zoom in to a point where your viewport is half as wide (on the ground), then new data will be requested. If you zoom back out so your viewport is twice as wide again (on the ground), data will again be requested. Setting a resFactor of 1 causes data to be requested with each change in resolution (this is equivalent to the behavior of the current WFS layer).

Attachments

resFactor.patch Download (3.6 KB) - added by tschaub 22 months ago.
add a resFactor option to make strategy more agressive
strat.patch Download (3.8 KB) - added by crschmidt 18 months ago.

Change History

Changed 22 months ago by tschaub

add a resFactor option to make strategy more agressive

Changed 18 months ago by igrcic

Confirming that is working just fine. Pretty useful.

Cheers, Ivan

Changed 18 months ago by crschmidt

  • state set to Review

Changed 18 months ago by elemoine

  • state changed from Review to Needs More Work

In the incomplete test this patch includes Tim mentions a bug with the "moveend" end. This needs clarification.

Changed 18 months ago by crschmidt

(In [9020]) Add/Update BBOX + resFactor patch (See #1830). Not entirely clear what the moveend event issue previously mentioned in the docs was for, but seems to be working fine for this test case.

Changed 18 months ago by crschmidt

Changed 18 months ago by crschmidt

  • state changed from Needs More Work to Review

New patch does the same as the old, but has some tests. Not entirely sure what the problem tim mentioned in the previous patch was, but I didn't seem to run into it here.

Changed 18 months ago by crschmidt

  • owner changed from fredj to crschmidt
  • status changed from new to assigned
  • state changed from Review to Commit

Tim and I are both happy with this; I'll let him commit

Changed 18 months ago by tschaub

I think the "moveend" issue was treated in #1831.

Changed 18 months ago by tschaub

  • status changed from assigned to closed
  • state changed from Commit to Complete
  • resolution set to fixed

(In [9087]) Adding an option to make the bbox strategy more agressive. Thanks for completing the tests crschmidt. r=crschmidt,igrcic (closes #1830)

Note: See TracTickets for help on using tickets.