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

Implement and test edge mode for WcsGeom.get_coord #1578

Closed
cdeil opened this issue Jul 27, 2018 · 2 comments
Closed

Implement and test edge mode for WcsGeom.get_coord #1578

cdeil opened this issue Jul 27, 2018 · 2 comments
Assignees
Labels
Milestone

Comments

@cdeil
Copy link
Contributor

cdeil commented Jul 27, 2018

In #1573 apparently @adonath tried to implement mode="edge" for WcsGeom.get_coord, see his edits here: https://github.com/gammapy/gammapy/pull/1573/files#diff-3cf936d0601fe08b95aa7b11fb9cf614

This doesn't work yet, it causes test fails in gammapy.maps:
https://ci.appveyor.com/project/cdeil/gammapy/build/1.0.3897/job/04tkynv5dibs2jbq#L2313

The missing mode="edge" for WcsGeom.get_coord was already noticed here:

# TODO: Improve by exposing a mode 'edge' for get_coord

I tried to do this in the past 10 min, but the code there is a bit complex.
So this is a reminder issue for myself to first implement mode="edge" for WcsGeom.get_coord with tests, and to adapt the solid_angle caller, and only then to finish up #1573 .

@cdeil cdeil added the bug label Jul 27, 2018
@cdeil cdeil added this to the 0.8 milestone Jul 27, 2018
@cdeil cdeil self-assigned this Jul 27, 2018
@cdeil cdeil added this to To do in gammapy.maps via automation Jul 27, 2018
@adonath
Copy link
Member

adonath commented Jul 30, 2018

As I'd like to finish up #1573, I'll take this over. Re-assigning to me...

@adonath adonath assigned adonath and unassigned cdeil Jul 30, 2018
@adonath adonath closed this as completed Jul 30, 2018
gammapy.maps automation moved this from To do to Done Jul 30, 2018
@adonath
Copy link
Member

adonath commented Jul 30, 2018

Resolved in #1592

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
gammapy.maps
  
Done
Development

No branches or pull requests

2 participants