<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jul 9, 2013 at 9:56 AM, Chris Harris <span dir="ltr"><<a href="mailto:chris.harris@kitware.com" target="_blank">chris.harris@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I am still able to travel through the map and get  a reversed image,<br>
is that by design?<br></blockquote><div><br></div><div style>like I said, the going though the map is not the problem introduced by this branch (it was before). The problem I fixed is what you reported</div><div style>where when you look at it from behind you were not getting proper tiles. I am not in favor of fixing the zoom in bug in this branch as its not introduced by this one. </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I have spotted one other issue, shouldn't prevent you merging, but I<br>
think the re-tiling needs to happen earlier in the zoom, see the<br>
attached screen shot, its very blurred?<br></blockquote><div><br></div><div style>That's just the zoom level computation (one line fix). I was planning to bump it up anyways. Will fix it. </div><div style><br></div><div style>
- Aashish</div><div style><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
May be we can have a look when you come in?<br>
<br>
Chris<br>
<br>
On Mon, Jul 8, 2013 at 6:36 PM, Aashish Chaudhary<br>
<div class="HOEnZb"><div class="h5"><<a href="mailto:aashish.chaudhary@kitware.com">aashish.chaudhary@kitware.com</a>> wrote:<br>
> Chris,<br>
><br>
> I was able to find a simple fix and pushed on both repositories. Make sure<br>
> you update your local copy before you try it.<br>
><br>
> Thanks,<br>
><br>
><br>
><br>
> On Mon, Jul 8, 2013 at 3:58 PM, Aashish Chaudhary<br>
> <<a href="mailto:aashish.chaudhary@kitware.com">aashish.chaudhary@kitware.com</a>> wrote:<br>
>><br>
>> Sorry forgot to answer one question:<br>
>><br>
>> I will have a look to see if there is  a simple fix but that may not<br>
>> happen until 4-4:30.<br>
>><br>
>> Thanks<br>
>><br>
>> - Aashish<br>
>><br>
>><br>
>> On Mon, Jul 8, 2013 at 3:55 PM, Aashish Chaudhary<br>
>> <<a href="mailto:aashish.chaudhary@kitware.com">aashish.chaudhary@kitware.com</a>> wrote:<br>
>>><br>
>>> On Mon, Jul 8, 2013 at 3:49 PM, Chris Harris <<a href="mailto:chris.harris@kitware.com">chris.harris@kitware.com</a>><br>
>>> wrote:<br>
>>>><br>
>>>> I am just working on chemistry dashboard issue and then I will get back<br>
>>>> to the review. However, I'm not sure it should go in with the zoom as it is?<br>
>>>> Is it simple to fix?<br>
>>><br>
>>><br>
>>> The zoom itself is not related to this (the branch didn't do anything for<br>
>>> it). I am not sure if I would like to fix the zoom going past the layer as<br>
>>> part of this branch primarily because<br>
>>><br>
>>> 1. The branch has lots of changes and I would like to get it in<br>
>>> 2. More importantly we should not allow interactions to go past the map<br>
>>> layer (what does that even mean?) So I am not too much worry about handling<br>
>>> that case for OSM (unless you have strong objections to it)<br>
>>><br>
>>> - Aashish<br>
>>><br>
>>><br>
>>><br>
>>>><br>
>>>><br>
>>>> Chris<br>
>>>><br>
>>>><br>
>>>> On Mon, Jul 8, 2013 at 3:46 PM, Aashish Chaudhary<br>
>>>> <<a href="mailto:aashish.chaudhary@kitware.com">aashish.chaudhary@kitware.com</a>> wrote:<br>
>>>>><br>
>>>>> Cool. So I guess you will get back to me on this in next hour or so?<br>
>>>>> (basically +1, -1 approval ratings)?<br>
>>>>><br>
>>>>> - Aashish<br>
>>>>><br>
>>>>><br>
>>>>> On Mon, Jul 8, 2013 at 3:07 PM, Chris Harris <<a href="mailto:chris.harris@kitware.com">chris.harris@kitware.com</a>><br>
>>>>> wrote:<br>
>>>>>><br>
>>>>>> Excellent!<br>
>>>>>><br>
>>>>>><br>
>>>>>> On Mon, Jul 8, 2013 at 3:02 PM, Aashish Chaudhary<br>
>>>>>> <<a href="mailto:aashish.chaudhary@kitware.com">aashish.chaudhary@kitware.com</a>> wrote:<br>
>>>>>>><br>
>>>>>>> Brad increased the size to 2MB. I pushed the branch on origin as<br>
>>>>>>> well.<br>
>>>>>>><br>
>>>>>>><br>
>>>>>>> On Mon, Jul 8, 2013 at 2:26 PM, Aashish Chaudhary<br>
>>>>>>> <<a href="mailto:aashish.chaudhary@kitware.com">aashish.chaudhary@kitware.com</a>> wrote:<br>
>>>>>>>><br>
>>>>>>>> Currently, if you go behind the map when zoom-in (which I will fix<br>
>>>>>>>> in a separate branch where you won't be able to do that), then the code may<br>
>>>>>>>> break.<br>
>>>>>>>><br>
>>>>>>>> Make sure that you don't go past the map layer (which could be<br>
>>>>>>>> tricky sometimes as it zoomin quite fast).<br>
>>>>>>>><br>
>>>>>>>> - Aashish<br>
>>>>>>>><br>
>>>>>>>><br>
>>>>>>>><br>
>>>>>>>> On Mon, Jul 8, 2013 at 2:23 PM, Chris Harris<br>
>>>>>>>> <<a href="mailto:chris.harris@kitware.com">chris.harris@kitware.com</a>> wrote:<br>
>>>>>>>>><br>
>>>>>>>>><br>
>>>>>>>>><br>
>>>>>>>>> On Mon, Jul 8, 2013 at 12:01 PM, Aashish Chaudhary<br>
>>>>>>>>> <<a href="mailto:aashish.chaudhary@kitware.com">aashish.chaudhary@kitware.com</a>> wrote:<br>
>>>>>>>>> > Thanks Chris. Couple of points to remember:<br>
>>>>>>>>> ><br>
>>>>>>>>> > 1. I kept the boundary lines (would be nice to turn it off when<br>
>>>>>>>>> > zoomed in<br>
>>>>>>>>> > close as these are low res) for now .<br>
>>>>>>>>> > 2. The zoom computation could be tweaked.<br>
>>>>>>>>><br>
>>>>>>>>> Yep, It looks like you can zoom through the map and then the tiling<br>
>>>>>>>>> stops working? And you end up with something like this?<br>
>>>>>>>>><br>
>>>>>>>>><br>
>>>>>>>>><br>
>>>>>>>>> > 3. After you +1 the branch, I would like to fix the coding style<br>
>>>>>>>>> > for newly<br>
>>>>>>>>> > added files (most of them are good)<br>
>>>>>>>>> ><br>
>>>>>>>>> > Thanks<br>
>>>>>>>>> ><br>
>>>>>>>>> ><br>
>>>>>>>>> ><br>
>>>>>>>>> > On Mon, Jul 8, 2013 at 11:51 AM, Chris Harris<br>
>>>>>>>>> > <<a href="mailto:chris.harris@kitware.com">chris.harris@kitware.com</a>><br>
>>>>>>>>> > wrote:<br>
>>>>>>>>> >><br>
>>>>>>>>> >> I will be happy to test is out for you.<br>
>>>>>>>>> >><br>
>>>>>>>>> >> Chris<br>
>>>>>>>>> >><br>
>>>>>>>>> >> On Mon, Jul 8, 2013 at 11:48 AM, Aashish Chaudhary<br>
>>>>>>>>> >> <<a href="mailto:aashish.chaudhary@kitware.com">aashish.chaudhary@kitware.com</a>> wrote:<br>
>>>>>>>>> >> > This branch has quite a few of changes and I would like<br>
>>>>>>>>> >> > someone else<br>
>>>>>>>>> >> > (Chris<br>
>>>>>>>>> >> > or Dave probably) to run a quick test on it.<br>
>>>>>>>>> >> ><br>
>>>>>>>>> >> > Summary:<br>
>>>>>>>>> >> > - Adds ability to get tiles from OSM<br>
>>>>>>>>> >> > - Updated and cleaned API<br>
>>>>>>>>> >> > - Basic work for projection<br>
>>>>>>>>> >> > - Added code for bin based sorting<br>
>>>>>>>>> >> ><br>
>>>>>>>>> >> > Some of the things are not perfect (like projection when I am<br>
>>>>>>>>> >> > not<br>
>>>>>>>>> >> > multiplying mecator with scaling) but that could be taken care<br>
>>>>>>>>> >> > later.<br>
>>>>>>>>> >> ><br>
>>>>>>>>> >> > @Dave do you have sometime this afternoon? I have a question<br>
>>>>>>>>> >> > on colop<br>
>>>>>>>>> >> > map.<br>
>>>>>>>>> >> ><br>
>>>>>>>>> >> > --<br>
>>>>>>>>> >> > | Aashish Chaudhary<br>
>>>>>>>>> >> > | R&D Engineer<br>
>>>>>>>>> >> > | Kitware Inc.<br>
>>>>>>>>> >> > | <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>>>>>>>> >> ><br>
>>>>>>>>> >> > _______________________________________________<br>
>>>>>>>>> >> > Opengeoscience-developers mailing list<br>
>>>>>>>>> >> > <a href="mailto:Opengeoscience-developers@public.kitware.com">Opengeoscience-developers@public.kitware.com</a><br>
>>>>>>>>> >> ><br>
>>>>>>>>> >> ><br>
>>>>>>>>> >> > <a href="http://public.kitware.com/cgi-bin/mailman/listinfo/opengeoscience-developers" target="_blank">http://public.kitware.com/cgi-bin/mailman/listinfo/opengeoscience-developers</a><br>

>>>>>>>>> >> ><br>
>>>>>>>>> ><br>
>>>>>>>>> ><br>
>>>>>>>>> ><br>
>>>>>>>>> ><br>
>>>>>>>>> > --<br>
>>>>>>>>> > | Aashish Chaudhary<br>
>>>>>>>>> > | R&D Engineer<br>
>>>>>>>>> > | Kitware Inc.<br>
>>>>>>>>> > | <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>>>>>>>><br>
>>>>>>>><br>
>>>>>>>><br>
>>>>>>>><br>
>>>>>>>> --<br>
>>>>>>>> | Aashish Chaudhary<br>
>>>>>>>> | R&D Engineer<br>
>>>>>>>> | Kitware Inc.<br>
>>>>>>>> | <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>>>>>><br>
>>>>>>><br>
>>>>>>><br>
>>>>>>><br>
>>>>>>> --<br>
>>>>>>> | Aashish Chaudhary<br>
>>>>>>> | R&D Engineer<br>
>>>>>>> | Kitware Inc.<br>
>>>>>>> | <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>>>>><br>
>>>>>><br>
>>>>><br>
>>>>><br>
>>>>><br>
>>>>> --<br>
>>>>> | Aashish Chaudhary<br>
>>>>> | R&D Engineer<br>
>>>>> | Kitware Inc.<br>
>>>>> | <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>>><br>
>>>><br>
>>><br>
>>><br>
>>><br>
>>> --<br>
>>> | Aashish Chaudhary<br>
>>> | R&D Engineer<br>
>>> | Kitware Inc.<br>
>>> | <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>><br>
>><br>
>><br>
>><br>
>> --<br>
>> | Aashish Chaudhary<br>
>> | R&D Engineer<br>
>> | Kitware Inc.<br>
>> | <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
><br>
><br>
><br>
><br>
> --<br>
> | Aashish Chaudhary<br>
> | R&D Engineer<br>
> | Kitware Inc.<br>
> | <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>| Aashish Chaudhary <br>| R&D Engineer         <br>| Kitware Inc.            <br>| <a href="http://www.kitware.com">www.kitware.com</a>    
</div></div>