fatiando / boule

Reference ellipsoids for geodesy and geophysics
https://www.fatiando.org/boule
BSD 3-Clause "New" or "Revised" License
38 stars 17 forks source link

Start unifying ellipsoids into a single class #77

Closed dabiged closed 2 years ago

dabiged commented 3 years ago

This PR unifies all ellipsoids (Spheres, oblate spheroids and triaxial ellipsoids) into a single class.

For discussion see #76

Reminders:

leouieda commented 3 years ago

@dabiged thanks for starting this 🙂 I would suggest writing the tests first and then working on the implementation. We already have most of the tests for oblate ellipsoids and spheres and those should all pass with the new class.

For this PR, it might be best to omit anything for triaxial ellipsoids (raising NotImplementedError). This will already be a lot of work without having to figure out new tests for the triaxial part. We can then work on that in #72

I'm having a loot at what we can do with attrs to figure out the axis values from the flattening. I'll push some changes to this PR when I have a bit of time.

leouieda commented 2 years ago

@dabiged thanks for your work on this and sorry for letting this stall. I'm merging this into the dev branch as is so that it's registered in the history as your contributing. We'll continue to work on this in there until it's done and then merge into master.

Thanks again for all you've done in Boule and I hope we can restart this at some point 🙂

dabiged commented 2 years ago

Thanks for the update. I have a job now that isn't in the geophysics space so I will see when I can get back to this!

leouieda commented 2 years ago

@dabiged congrats on the new job 🎉 No rush and thanks for all you did already