Amit Aronovitch

continuing discussion from #262 - Actually, I now see that my comment [there](https://github.com/satellogic/telluric/pull/262#issuecomment-576799077) was mistaken - it appears that _raster_opener is NOT a context manager, but in fact just a...

We (me and @moshe742) looked at this, and it seems that the reason it does not work is because mercator is limited in latitude so cannot hold the vector. However,...

> `_raster_opener` can leak memory, since it never closes the object. we need to solve that - do you have a fix? (this PR is good, but seem unrelated to...

I think if you just add @contextlib.contextmanager above the original function, and replace the "return" with a "yield" then it should work. On the other hand I think my solution...

BTW, sorry @maralonso - when I wrote my original response above - I did not notice that the original `save_cloud_optimized` implementation had actually called _raster_opener, so I just failed to...

BTW: If I disable the browser test completely, it works fine on Firefox as well. Maybe we can remove it entirely, and replace it with a non-blocking recommendation instead? ("Unsupported...

update: looks like @shiny-mountain already has a fix in staging branch, thanks! :ok_hand:

ping... Made a new version of this fix #40 @sgillies : implemented there as @tmcw suggested, and also added test. Can/should we revive this project? Or is there already some...

some update which might be related (I am not sure that it would solve the problem for the OP) TLDR: disabling IPV6 for the probematic connection (via NetworkManager UI) seems...