nodrogluap / OpenDBA

GPU-accelerated Dynamic Time Warp (DTW) Barycenter Averaging
Other
64 stars 13 forks source link

Compilation issue with `make tests` #14

Closed liam-mitchell21 closed 2 years ago

liam-mitchell21 commented 2 years ago

Hello,

When I clone the repo, make works fine, but make tests seems to run into a linking problem. Problem in question

I tried fiddling around with the include lines inside io_test_utils.cu, but changing the includes paths resulted in 'no such file or directory' errors, so it seems that io_test_utils.cu can see the included io_utils.hpp and cpu_utils.hpp fine, it just isn't importing the functions properly. I also tried appending export to the function definitions, and adding -x cu to the compiler call, but neither of these solved it.

I'm not too familiar with C++, so it's likely I'm missing something simple, I look forward to learning what it is. --Liam

nodrogluap commented 2 years ago

The tests assume that you've compiled (make) with HDF5_SUPPORTED=1. This is probably a bad assumption, but not 100% sure how I'd ensure the same env setting was used for both make and make tests, unless I make them one target rather than two. Lemme think about that. In the meantime, can you confirm that if you run "make HDF5_SUPPORTED=1" on a clean version, that "make tests" then works.

nodrogluap commented 2 years ago

As a start, I've changed the README's Quick Start section to distinguish between how you use the make commands with and without HDF5 support (i.e. if you plan to use it for nanopore data or not).

nodrogluap commented 2 years ago

Commit to master dd826bf fixes assumption of HDF5 support. Tested with and without HDF5_SUPPORTED=1 flag in make command. Tentatively closing, feel free to reopen if this doesn't work in your hands.

liam-mitchell21 commented 2 years ago

Yes, running make HDF5_SUPPORTED=1 &&​ make HDF5_SUPPORTED=1 tests​ seemed to work fine, thanks for the help!

--Liam


From: Paul Gordon @.> Sent: June 12, 2022 9:51 PM To: nodrogluap/OpenDBA @.> Cc: Mitchell, Liam @.>; Author @.> Subject: Re: [nodrogluap/OpenDBA] Compilation issue with make tests (Issue #14)

Caution: This email was sent from someone outside of the University of Lethbridge. Do not click on links or open attachments unless you know they are safe. Suspicious emails should be forwarded to @.***

Closed #14https://github.com/nodrogluap/OpenDBA/issues/14 as completed.

— Reply to this email directly, view it on GitHubhttps://github.com/nodrogluap/OpenDBA/issues/14#event-6792753289, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH3DR4GYZVAKAKMEG7FWXSTVO2V3JANCNFSM5YSXFMJQ. You are receiving this because you authored the thread.Message ID: @.***>