WillAyd commented on PR #484: URL: https://github.com/apache/arrow-nanoarrow/pull/484#issuecomment-2164177610
Sweet thanks for giving this a shot > Metal C++ is its own thing and is a sort of header-only wrapper around the Objective C (not included directly in the framework): Possibly, though I would still hope Meson could handle that. I found another project in the WrapDB that uses Metal: https://github.com/mesonbuild/wrapdb/blob/2d9ed3ad5bed879d8aa010de21ae165f6cf5ce4d/subprojects/packagefiles/imgui/meson.build#L49 Maybe we just need to add `required: get_option('metal')` to our dependency call? > ...and this is maybe because `cublas` is not the right thing to link to (the thing we're using is called the "driver" API, which is _not_ the "runtime" API). Also possibly because there needs to be a `-DNANOARROW_WITH_CUDA` compile define to get the `ArrowDeviceCuda` symbol into `nanoarrow_device.so` (still workshopping the final version of that). Ah OK...apologies as I know embarassingly little about CUDA. But in the configuration now I have `modules: [cublas]` as I just copy-pasted that from the Meson documentation without much thought. Maybe that needs to be `modules: [cuda-driver]`? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
