Skip to content
New issue

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

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

Already on GitHub? # to your account

python3 compatibility? #6

Open
proximous opened this issue Apr 28, 2016 · 2 comments
Open

python3 compatibility? #6

proximous opened this issue Apr 28, 2016 · 2 comments

Comments

@proximous
Copy link

Not sure how much of a problem it is, but line 267 in geomag.py should explicity use integer division ('//') to be python3 compatible.

$ python -3                                               
Python 2.7.6 (default, Jun 22 2015, 17:58:13) 
[GCC 4.8.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import geomag
/usr/local/lib/python2.7/dist-packages/geomag/geomag.py:267: DeprecationWarning: classic int division
  D2=(n-m+D1)/D1

the compatible version would be:

  D2=(n-m+D1)//D1

(double slash to indicate integer division)

@todd-dembrey
Copy link

Would be fixed by #3

@sposs
Copy link

sposs commented Jan 6, 2020

Hi,
I'm facing an error using the 'public' package of the package:

    __singleton__ = GeoMag()
NameError: name 'GeoMag' is not defined

I believe the issue does not occur with the code from @todd-dembrey. Is this repo still active? If yes, can those changes be merged?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants