geomag icon indicating copy to clipboard operation
geomag copied to clipboard

Reworked Geomag

Open todd-dembrey opened this issue 8 years ago • 7 comments

Major rework, changes include:

  • rewrite of method
  • change of names to better reflect process
  • creation of documentation
  • additional testing
  • example

Works with both 2/3 python and backwards compatible.

Happy to discuss further changes if required?

todd-dembrey avatar Oct 28 '15 22:10 todd-dembrey

I would leave the _build directory of the doc out of the repository, leave the client make the doc if needed.

sposs avatar Oct 03 '16 15:10 sposs

@sposs I think I have included some of the changes from #7 in my enthusiasm. Thanks for the review.

todd-dembrey avatar Oct 08 '16 14:10 todd-dembrey

It's a big +1 for me. Great improvement!!

sposs avatar Oct 08 '16 15:10 sposs

Any chance of this getting merged at some point? PR is a fantastic rewrite - some colleagues lost a few hours working with master before realizing that Todd's fork existed and was more up to date. Otherwise, pointing people to todd-dembrey/geomag in a README might be helpful.

rprechelt avatar Dec 01 '16 09:12 rprechelt

@cmweiss I would also like to start working with the rewritten branch - are you still interested in maintaining this repository or would you like to hand it off (with credit of course) so development can continue?

slibby avatar Sep 22 '17 23:09 slibby

@cmweiss @todd-dembrey Any update on this? I'm using Django 2.0.4, which requires Python 3.

singlerider avatar Apr 14 '18 00:04 singlerider

I don't think there is any Python 2 specific code. It should work with Python 3.

cmweiss avatar Apr 14 '18 15:04 cmweiss