bistromath / gr-ais

Automatic Information System decoder for shipborne position reporting for the Gnuradio project
134 stars 47 forks source link

Python naming conflict with libais #27

Closed vtmmm closed 3 years ago

vtmmm commented 3 years ago

The name of the Python module is simply "ais"; this conflicts with libais, which can parse AIS messages using a Python wrapper.

I'd like to use both gr-ais and libais using GR3.8 and Python 3.8+, as they are complimentary, but am running into issues trying to import them both.

I'm currently reading up on the correct way to import Python modules by path and reading up on/trying to use the importlib API, but if someone has already solved this issue I'm all ears. I also wanted to bring it up in case refactoring this module to use a less generic name could make sense.

bistromath commented 3 years ago

I'm refactoring gr-ais for GR 3.9, and I'll try to figure out how to get them to coexist. Unfortunately, Python has adopted the Highlander model of package naming -- there can be only one.

On Wed, Feb 10, 2021 at 11:21 PM vtmmm notifications@github.com wrote:

The name of the Python module is simply "ais"; this conflicts with libais https://github.com/schwehr/libais, which can parse AIS messages using a Python wrapper.

I'd like to use both gr-ais and libais using GR3.8 and Python 3.8+, as they are complimentary, but am running into issues trying to import them both.

I'm currently reading up on the correct way to import Python modules by path and reading up on/trying to use the importlib API, but if someone has already solved this issue I'm all ears. I also wanted to bring it up in case refactoring this module to use a less generic name could make sense.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/bistromath/gr-ais/issues/27, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADBTK37U22372XEKXCJGZTS6OAQXANCNFSM4XOKWKHA .

vtmmm commented 3 years ago

Okay, good deal!

bistromath commented 3 years ago

Fixed in 78e441.