Re^5: Inline::CUDA : looking for feedback

There's 15 failed tests ("Failed 15/20 test programs."): I'll need to go through each to find the actual problems.

If all are simply quoting uses, then the solution is straightforward (if a little tedious to implement across the board). Let's see how that pans out before worrying about config modifications and the like.

I recall you specified a minimum Perl version of v5.6. I've checked https://perldoc.perl.org/5.6.0/functions/quotemeta and https://perldoc.perl.org/5.6.0/perlop; quotemeta, \Q and \E are all available in that version, so no issues there.

"If all else fails I will have to resort to prompting the user during installation. Any opinions on that style?"

As I said, let's wait and see if secondary options are necessary; however, my general "opinions on that style" are:

I don't, as a rule, like it. On a number of occasions, I've started what I expected to be a fairly long-running installation, gone off to do something else while it runs, then returned ten or so minutes later only to find that, two seconds after I walked away from the screen, it prompted: "Specify location of whatever [/usr/bin/whatever]?" or something similar.
There are cases where it simply can't be avoided.
I have (reluctantly) used it myself in production code.
Where it has been necessary, I have tried to indicate early on that user intervention will be required.

"Thanks for your time spent and valuable feedback. I am not in a hurry, please work on it in your own pace."

No worries. I am rather eager to see how this works.

I have, in fact, spent most of this week (at $work) modifying modules and repeatedly going through the make cycle, tweaking tests to reflect the changes. While I'm happy to do this, "All work and no play makes Ken a dull boy.", so I need a break. As I said this morning, I can probably look into this further over the weekend. [Current lockdown restrictions means that I'm not even allowed guests in my home, so I'm certainly free to do this work.]

— Ken

Comment on Re^5: Inline::CUDA : looking for feedback Select or Download Code

Replies are listed 'Best First'.
Re^6: Inline::CUDA : looking for feedback by bliako (Abbot) on Jul 30, 2021 at 12:33 UTC
I have updated the github repository with a new version which uses ShellQuote::Any to do (some of) the quoting. I have also added a test which tries to run some perl-scripts with space in their name. That's in `t/002-spaces-spaces-spaces.t`. I have also added another repository: https://github.com/hadjiprocopis/perl-nvidia2-ml which takes over nvidia::ml, fixes its compilation errors and adds 1 more binding. If you install this first, it will be easier to enquire the GPU in current module. It requires nvidia-ml binary library which must be in the toolkit you downloaded. If you can't install that then it's not a problem but 1 test will fail (`t/001-enquire-nvidia-gpu.t`). I suggest you run `perl Makefile.PL` and if successful, then inspect `config/Inline-CUDA.conf` to see if `cc, cxx, nvcc, ld` are all set to valid names (unquoted). Following that, run just a few tests: `t/000-require-modules.t`, `t/002-spaces-spaces-spaces.t`, `t/008-config.t`, `t/011-basic.t` . The last two are compiling CUDA code. If all goes well, then it's likely that all tests will succeed. `031-matrix-multiplication-pass-parameters.t` does a matrix multiplication with data passed from the Perl script and results returned back. That's a good base for any CUDA code you may want to write. The prompting during installation is only for windows, the user must somehow indicate where a c-compiler is if not in path already. BTW, you will be using a newer version of nvcc than mine. So a lot can go wrong there too. bw, bliako	[reply] [d/l] [select]
Re^7: Inline::CUDA : looking for feedback by kcott (Archbishop) on Jul 31, 2021 at 01:42 UTC
"I suggest you run `perl Makefile.PL` ..." $ perl Makefile.PL PACK001=HASH(0x800172c08) Changes PACK001=HASH(0x800172c08) ignore.txt PACK001=HASH(0x800172c08) lib PACK001=HASH(0x800172c08) nvidia-ml-pl.pdf PACK001=HASH(0x800172c08) nvml.h PACK001=HASH(0x800172c08) nvml_wrap.c PACK001=HASH(0x800172c08) README PACK001=HASH(0x800172c08) README.md PACK001=HASH(0x800172c08) xt PACK001=HASH(0x800172c08) . PACK001=HASH(0x800172c08) nvidia2 PACK001=HASH(0x800172c08) blib/lib/nvidia2/ml.pm PACK001=HASH(0x800172c08) blib/lib/nvidia2/smi.pm PACK001=HASH(0x800172c08) ml PACK001=HASH(0x800172c08) blib/lib/nvidia2/ml/bindings.pm PACK001=HASH(0x800172c08) blib/lib/nvidia2/ml/Utils.pm Warning (mostly harmless): No library found for -lnvidia-ml Generating a Unix-style Makefile Writing Makefile for nvidia2::ml::bindings Writing MYMETA.yml and MYMETA.json [download] "... and if successful, then inspect `config/Inline-CUDA.conf` ..." Couldn't find `Inline-CUDA.conf` even with a case-insensitive search. I tried a `make` anyway: $ make cp lib/nvidia2/ml.pm blib/lib/nvidia2/ml.pm cp lib/nvidia2/ml/Utils.pm blib/lib/nvidia2/ml/Utils.pm cp lib/nvidia2/ml/bindings.pm blib/lib/nvidia2/ml/bindings.pm cp lib/nvidia2/smi.pm blib/lib/nvidia2/smi.pm Running Mkbootstrap for bindings () chmod 644 "bindings.bs" "/home/ken/perl5/perlbrew/perls/perl-5.34.0/bin/perl.exe" -MExtUtils:: +Command::MM -e 'cp_nonempty' -- bindings.bs blib/arch/auto/nvidia2/ml +/bindings/bindings.bs 644 gcc -c -I. -DPERL_USE_SAFE_PUTENV -U__STRICT_ANSI__ -D_GNU_SOURCE -fw +rapv -fno-strict-aliasing -pipe -fstack-protector-strong -D_FORTIFY_S +OURCE=2 -DUSEIMPORTLIB -O3 -DVERSION=\"5.00\" -DXS_VERSION=\"5.00\" + "-I/home/ken/perl5/perlbrew/perls/perl-5.34.0/lib/5.34.0/cygwin-thr +ead-multi/CORE" nvml_wrap.c rm -f blib/arch/auto/nvidia2/ml/bindings/bindings.dll g++ --shared -Wl,--enable-auto-import -Wl,--export-all-symbols -Wl,- +-enable-auto-image-base -L/usr/local/lib -fstack-protector-strong nv +ml_wrap.o -o blib/arch/auto/nvidia2/ml/bindings/bindings.dll \ /home/ken/perl5/perlbrew/perls/perl-5.34.0/lib/5.34.0/cygwin-thread- +multi/CORE/cygperl5_34_0.dll -lpthread -ldl \ /usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: +nvml_wrap.o:nvml_wrap.c:(.text+0x6af5): undefined reference to `nvmlI +nit' /usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: +nvml_wrap.o:nvml_wrap.c:(.text+0x6c05): undefined reference to `nvmlS +hutdown' ... several screenfuls of lines like that ... /usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: +nvml_wrap.o:nvml_wrap.c:(.text+0x1e977): undefined reference to `nvml +DeviceGetSupportedMemoryClocks' /usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: +nvml_wrap.o:nvml_wrap.c:(.text+0x1e9ca): undefined reference to `nvml +DeviceGetSupportedGraphicsClocks' collect2: error: ld returned 1 exit status make: *** [Makefile:482: blib/arch/auto/nvidia2/ml/bindings/bindings.d +ll] Error 1 [download] — Ken	[reply] [d/l] [select]
Re^8: Inline::CUDA : looking for feedback by Anonymous Monk on Jul 31, 2021 at 09:17 UTC
nvidia2-ml's Makefile.PL contains the directive to link to nvida-ml dll . Spexify there the dir this library resides prepending a `-L adir` use quotes if spaces in adir. re: the config file etc., i was referring to Inline::CUDA. let me know bliako	[reply] [d/l]