-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Compile issues #3
Comments
Actually scratch that I solved it with sudo apt install libzstd-dev... there are some undocumented dependencies that need putting on this repo: I'll add more as I get through the process |
I get this when running this command:
lib.h doesn't seem to exist anywhere in the repository or code, other than in GPUDecoder.cpp. |
@vellamike I should provide more details. I’m compiling on a new Nvidia Jetson Orin development kit. |
Hi - this header is part of the koi pre-built library which should be downloaded as part of the cmake configuration process. Please could you attach the output from the |
Ah, this will explain the problem. Arm Linux platforms are not currently supported. |
Okay next issue fighting with CUDA (that's fixed now), but now I am getting:
|
i.e. I meant another dependency is you need nvcc 11. If you install cuda from the Ubuntu repository you get v10 |
The dorado/3rdparty/hdf_plugins folder is a submodule which should have been populated during the |
No, there's nothing in that folder:
|
It looks like none of the submodules are populated for some reason. It also looks like this is reconfiguring into an existing cmake-build folder. could you try removing the build artefact folder and then re-running:
|
Nope, same problem occurs... |
How about if you try to update the submodules manually: |
Before I got your comment I rm -rf'd the directory and started again and it compiled no probs. Will try running it now! Thanks for your help. |
One final Q - are there R10.4.1 and E8.2 models available for Dorado? Or do I just use the 10.4 / e8.1 models? |
There are no E8.2 models available at the moment, but we will be adding more run conditions soon. You can use |
Success - it's basecalling off the R10.4.1 data we have produced... I will report back |
First observation - blimey it's fast! Even in SAC mode |
@adbeggs Great to hear! I will add the kit 14 models for you tomorrow. |
@MarkBicknellONT any plans for the Linux Arm libraries? 🤞🏻Happy to work with you to get it up and running. |
@mcrone we have some Orin devkits and will take a look for you next week. |
I'm getting some errors while compiling (solved a lot of others btw, by loading the required packages on our HPC) `
Some of these libraries may not be found correctly. WARNING: Target "dorado_tests" requests linking to directory "/projects/0/lwc2020006/software/nanoporetech/dorado/dorado/3rdparty/fake_cudnn". Targets may link only to libraries. CMake is dropping the item.
Some of these libraries may not be found correctly. -- Generating done any idea if this affects the final build? |
@vdejager Can you give details of the system (OS & architecture) you are building on. It looks like you are getting warnings but no errors? I believe your final build will be OK, though we do need to resolve these warnings. |
Hi Mike, I'm building on our HPC system with A100's available for GPU, which we also use with Bonito.
Currently I have to load the following modules (as expected) and commands to get a binary in the cmake-build/bin directory module load 2021 this does not work (paths taken from module show OpenSSL/1.1)#export OPENSSL_CRYPTO_LIBRARY=/sw/arch/Centos8/EB_production/2021/software/OpenSSL/1.1/lib64 original#cmake -S . -B cmake-build setting the libraries on the commandline workscmake -S . -B cmake-build -DOPENSSL_INCLUDE_DIR=/sw/arch/Centos8/EB_production/2021/software/OpenSSL/1.1/include/openssl -DOPENSSL_SSL_LIBRARY=/sw/arch/Centos8/EB_production/2021/software/OpenSSL/1.1/lib64/libssl.so -DOPENSSL_CRYPTO_LIBRARY=/sw/arch/Centos8/EB_production/2021/software/OpenSSL/1.1/lib64/libcrypto.so cmake --build cmake-build --config Release -- -j cmake --build produces a binary, but ctest just returns the prompt |
If CUDA is strictly needed, it should be mentioned in the dependencies in
|
@vdejager we haven't yet tested on CentOS - we are going to do this and update the README. Does It is odd that |
@gringer the CUDA dependency is already noted in the |
This is compiling for me now, so some of the changes over the past week must have fixed whatever the problem was. |
I think we've sorted all of the issues here now 🎉 |
The following crash was seen on CI when compiling with GCC 8.4.0 and with ASAN enabled: ==11805==ERROR: AddressSanitizer: SEGV on unknown address 0x001200000000 (pc 0xffff68c2296c bp 0xffffc1c19400 sp 0xffffc1c19400 T0) ==11805==The signal is caused by a WRITE memory access. #0 0xffff68c2296b (/lib/aarch64-linux-gnu/libc.so.6+0x8396b) #1 0xffff876bedc3 (/usr/lib/aarch64-linux-gnu/libasan.so.5+0x36dc3) #2 0xaaaad510039b in _GLOBAL__sub_I_00099_1__ZN6dorado8splitter7subreadERKNS_11SimplexReadESt8optionalISt4pairImmEES6_ (dorado_tests+0x17639b) #3 0xaaaad5c9ea1f in __libc_csu_init (dorado_tests+0xd14a1f) The static initialiser that calls into libasan is one of ASANs inserted init functions (__asan_init, __asan_version_mismatch_check_v8, or __asan_register_globals) though gdb refuses to ptrace so I can't tell which. None of them should crash so this feels like a compiler bug, though I'm unable to determine exactly what's triggering it since a disassembly of the static initialiser doesn't change between the broken and "fixed" versions of the code (plt addresses notwithstanding).
Trying to compile on the P24 (also have tried our A100 HPC). getting the following:
Any ideas? Initially it was complaining that it couldn't find hdf at all so I sudo apt install libhdf5-dev
Thanks
Andrew
The text was updated successfully, but these errors were encountered: