Skip to content
This repository has been archived by the owner on Jun 1, 2020. It is now read-only.

Pre-Rotation of co-ordinates #6

Open
steven-murray opened this issue Aug 14, 2017 · 2 comments
Open

Pre-Rotation of co-ordinates #6

steven-murray opened this issue Aug 14, 2017 · 2 comments

Comments

@steven-murray
Copy link
Contributor

I was wondering if it is considered a good idea to have the ability to pre-rotate the co-ordinates of the map?

If one requires output at many frequencies, and all of them to be rotated, then it is particularly inefficient to rotate the map at each frequency. However, if the 6 underlying maps are each rotated to begin with before interpolation, it is much more efficient. I have code that does this which I am currently using (and it seems to work quite well for my specific use-case). I can make a PR if you think it's something that would be useful.

@telegraphic
Copy link
Owner

Agreed that this may be more efficient for many use cases. If you are happy to try a PR please do go ahead.

I don't have much/any time over the next few months, but would you be interested in a 'hackathon' day later in the year to work on this + other features?

@steven-murray
Copy link
Contributor Author

Sorry for the delay... I've sent a pull request that makes these changes: #8

Let me know what you think.

I've been pretty busy for the latter half of the year... if you have some stuff you'd like to work on, let me know and I can do a day maybe towards the start of next year?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants