search

casacore / python-casacore

Python bindings for casacore, a library used in radio astronomy
http://casacore.github.io/python-casacore
GNU Lesser General Public License v3.0
36 stars 22 forks source link

Runtime linking not working with binary wheel distribution #171

Open tammojan opened 5 years ago

tammojan commented 5 years ago

The following command will fail with a binary distribution, since it relies on runtime linking:

import casacore.tables as pt
# Get the current azimuth and elevation of the sun in Dwingeloo
pt.taql("select meas.azel('Sun', datetime(), 'DWL')")

This requires libcasa_meas.so to be in the runtime linking path. This is not the case in the current wheel distribution prototype.

gijzelaerr commented 5 years ago

I guess this has been resolved by https://github.com/casacore/python-casacore/pull/177 ?

gervandiepen commented 5 years ago

Yes; in that time runtime linking is not necessary anymore for special TaQL functions. However, unknown storage managers (such as LofarStMan) cannot be loaded. Since they are part of other software, there is no way to preload them.

On Wed, Jun 5, 2019 at 12:10 PM Gijs Molenaar notifications@github.com wrote:

I guess this has been resolved by #177 https://github.com/casacore/python-casacore/pull/177 ?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/casacore/python-casacore/issues/171?email_source=notifications&email_token=AB34QPBDUPHUNCQUUFUN57TPY6GJFA5CNFSM4HOJDHKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW7H4WY#issuecomment-499023451, or mute the thread https://github.com/notifications/unsubscribe-auth/AB34QPDUPPRIBI7TFITNPV3PY6GJFANCNFSM4HOJDHKA .

gijzelaerr commented 5 years ago

I guess we can then close this issue when it is properly documented.

We can probably expect bug reports about this when we start distributing the binary wheels.