Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

boundsFromZoomAndCenter returned incorrect values. #625

Merged
merged 2 commits into from
Oct 13, 2016
Merged

Conversation

manthey
Copy link
Contributor

@manthey manthey commented Oct 13, 2016

When using any gcs other than the map's gcs, the function performed two conversions on the map center's coordinates which resulted in erroneous results.

When using any gcs other than the map's gcs, the function performed two conversions on the map center's coordinates which resulted in erroneous results.
@manthey
Copy link
Contributor Author

manthey commented Oct 13, 2016

@jbeezley This was the error that we ran into on the HistomicsTK project getting the image bounds.

@jbeezley
Copy link
Contributor

Probably yes. I'll have to check to make sure.

@codecov-io
Copy link

codecov-io commented Oct 13, 2016

Current coverage is 85.43% (diff: 100%)

Merging #625 into master will increase coverage by 0.02%

@@             master       #625   diff @@
==========================================
  Files            85         85          
  Lines          8217       8217          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           7018       7020     +2   
+ Misses         1199       1197     -2   
  Partials          0          0          

Powered by Codecov. Last update 470d78f...b78ba96

Copy link
Contributor

@jbeezley jbeezley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fixes the issue I was having with the bounds call. LGTM

@manthey manthey merged commit b6bf634 into master Oct 13, 2016
@manthey manthey deleted the fix-get-bounds branch October 13, 2016 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants