Marcello Seri

Results 435 comments of Marcello Seri

Yes, this seems doable indeed. I don’t have time to work on it at the moment, but I would be happy to review a PR fixing it

That will not be a problem as long as owl is still building fine with dune2

Now also these tests spot some of the linking issues: `# Error: Error on dynamically loaded library: /home/runner/.opam/4.10.1/lib/stublibs/dlleigen_cpp_stubs.so: /home/runner/.opam/4.10.1/lib/stublibs/dlleigen_cpp_stubs.so: undefined symbol: _ZTVN10__cxxabiv117__class_type_infoE`

Not working still. I am trying to figure out if and how to use the new dune feature but mixing c and cpp seems to be still a problem

I see. They use https://portal.nersc.gov/project/sparse/superlu/ as solver, it may well be that it is much more optimised. Afaik eigen does transform the sparse matrix to a dense matrix internally in...

Now that we are using `dune` this would be quite easy to fix: we could add a configurator step creating the right flags, or using ocaml syntax in the `dune`...

`foreign_library` does not support `c_library_flags`, that has to go in the library using the stubs. Looks like we have two issues which are mutually breaking with things as they are...

I am doing that locally in a branch, seems to work but still testing. But then I am quite confused by the difference between the compiler/preprocessor flags and the link...