Rectify Method is not saved

[lklepner]

I've noticed that the rectify method is not being saved - ff the page is reloaded the rectification approach returns to the default setting and if the map is cropped the default rectification method is used.

This is a problem because some maps yield much better results with non-default rectification types.

In the case of one of the Hudson River Atlas maps (http://maps.nypl.org/warper/maps/14911) the river is significantly compressed on the map to make room for additional inland space (see note overlaid on the river itself). Thin Plate Spline yields much better results in this case than the auto rectification setting... but the improvement will not be retained in the long run since a page refresh or crop operation will return the rectify method to the original setting.

Thin Plate Spline -

After reloading the page, allowing the rectify method to change back to auto -

[mgiraldo]

good catch. i'd say this is a bug and we need to fix that

[mgiraldo]

this is fixed now: http://maps.nypl.org/warper/maps/14911

[lklepner]

Maps are re-rectified when cropped. I tested that route and got the default/wrong result.

[mgiraldo]

I suppose you did not modify the select boxes? only pressed rectify in the rectify tab?

can you provide the steps to reproduce because I tried several times and this is the result I get after refreshing:

On May 12, 2016, at 19:22, Lou Klepner notifications@github.com wrote:

Maps are re-rectified when cropped. I tested that route and got the default/wrong result.

—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub

[mgiraldo]

oh I think I know what you're talking about

I will look into that. a new bug needs to be filled for that

• mga

On May 12, 2016, at 19:22, Lou Klepner notifications@github.com wrote:

Maps are re-rectified when cropped. I tested that route and got the default/wrong result.

—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub

[mgiraldo]

fixed in #51 👍

[lklepner]

confirmed - thanks!