delphi icon indicating copy to clipboard operation
delphi copied to clipboard

Cant use cargo +nightly build --release

Open watsonj8atwit opened this issue 1 year ago • 6 comments

Screenshot 2024-07-10 191928

watsonj8atwit avatar Jul 10 '24 23:07 watsonj8atwit

The issue is with the private_in_public lint.

You can remove the line that contains it in ~~rust/algebra/lib.rs~~. Edit: rust/algebra/src/lib.rs.

lib.rs file

some stuff...

#![deny(
    unused_must_use,
    unused_mut,
    unused_unsafe,
            private_in_public,  < < <  Remove this line :)  < < < < < < < < <
    unsafe_code
)]

some more stuff...

adomaitisc avatar Jul 21 '24 22:07 adomaitisc

i did remove that line and in my opinion i think not doing nightly rustup might fix it running a current version of rust should fix the issue did not test it yet though

watsonj8atwit avatar Jul 21 '24 23:07 watsonj8atwit

i remove the line last week and still had issues

watsonj8atwit avatar Jul 21 '24 23:07 watsonj8atwit

That line occurs in another file as well, then it asked for llvm config.

I managed to run the tests and benchmark, then it was possible to perform the inference with one of the models in the repository.

The tests and benchmark look really odd tho, but they aren't exiting with an error.

adomaitisc avatar Jul 22 '24 00:07 adomaitisc

I was able to follow the instructions up to the build and test phase. However, when integrating TensorBoard, running the model, and extracting weights, I encountered an error. I suspect that this GitHub repository hasn't been thoroughly tested with a fresh installation in the past two years, leading to compatibility issues. I appreciate the authors' initial work on this proof of concept but understand they might have moved on to other projects.

watsonj8atwit avatar Jul 22 '24 00:07 watsonj8atwit

o to fix llvm config do this LVM Configuration:

The warning about llvm-config --prefix indicates that the LLVM configuration is missing. You need to install LLVM and set the LLVM_CONFIG_PATH environment variable.

sudo apt-get install llvm export LLVM_CONFIG_PATH=$(which llvm-config)

Compiler Flags:

Check if the compiler flags or environment variables are set correctly. The missing CXX variable indicates the C++ compiler might not be properly specified. Set the appropriate compiler in your environment:

export CXX=g++

then when in delphi/rust do cargo clean then run build again

also make sure your memory as much and you can reasonable give it

watsonj8atwit avatar Jul 22 '24 01:07 watsonj8atwit