Open kovar-ursa opened 2 years ago
@kovar-ursa — Well, drat. When I run from source, everything works, and I've seen no errors like this in CI; but when I install the package with pip
, I get the same error you do! Let me dig in and try to find a difference.
I am mystified about how the 4.1.2 build produced compiled packages that are missing code present in the 4.1.2 version in the repository. Before investigating further, I'm attempting a new release, since a bugfix release is due anyway and might fix this "ha" problem at the same time. It's now building on GitHub and should self-publish to PyPI within about an hour. Once 4.1.3 is available, we can see if it's still missing the new "ha" attribute.
Thanks for letting me know!
Greetings, Happy to help. 'tis a most helpful package. I'm only using a very small piece of it, but it solves that piece perfectly.
Well, it didn't work. Version 4.1.3 also built successfully and passed all its tests, but when pip installed doesn't have the "ha" attribute on the Sun. Time for me to stare really hard at this problem and see if I can think of something.
I've lost hours, and even days, to this sort of thing. I'm with you in spirit!
It turns out my local test was a false positive! The test file was in a directory that contained an ephem/
directory inside of which was an older version of the library, that of course was throwing the error.
Once I moved into a fresh directory and did a fresh install, the new version worked fine.
Somehow, in a way I don't quite understand, an old binary for libastro was somehow getting imported at the same time as a Python __init__.py
that was clearly modern and wanted to use the new .ha
attribute.
So on my system, at least, the new 4.1.3 runs the test file above just fine. Could you double-check on your end? Thanks!
Greetings,
It works on the known use case!
I've several hundred thousand points so it may hiccup on something else. Will let you know if so.
Thanks very much!
-David
ephem is version 4.1.2 now
That code used to work (absent the previously reported bug.) It now produces the following for all cases, not just the edge case: