GeomScale / volesti

Practical volume computation and sampling in high dimensions
GNU Lesser General Public License v3.0
142 stars 114 forks source link

Error is generated while trying to run Volesti in RStudio of Win11 #272

Open Soumya624 opened 1 year ago

Soumya624 commented 1 year ago

Description Error is generated while trying to run Volesti in RStudio of Win11. Although bug is not appearing in Linux or WSL

Screenshots Screenshot20191555

vissarion commented 1 year ago

Could you try generating the CRAN package and try to install it? What happens if you try to install the CRAN from https://cran.r-project.org/web/packages/volesti/index.html See also https://github.com/GeomScale/volesti/issues/271#issuecomment-1475816023

Soumya624 commented 1 year ago

Hi @vissarion ,

Thanks for the previous link. After going through it, I found the problem was there due to lp_solve library.

Screenshot 2023-03-28 021708

Even after trying install.packages("lpSolve"), it was storing somewhere else. So, fixed the issue by changing the Makevars.win file.

Basically, added PKG_LIBS = -L"C:/Program Files/lp_solve_5.5/" -llpsolve55, in that file. And replaced the "C:/Program Files/lp_solve_5.5/" with the actual path to the directory where the lp_solve library is installed.

Then installed the volesti package again using the command devtools::install_github(...)

Soumya624 commented 1 year ago

Should i update this in a PR? Also would like to work on the idea "Move R interface of volesti to a new repository", if it's still vacant. I think issues will be less if we start building a separate repository for volesti_R!

vissarion commented 1 year ago

It is not clear that the issue is general and not an issue of your environment, since the package is building correctly in several windows platforms (see github actions and CRAN tests). Is your Rstudio version and/or windows version different than the ones in CI tests?

Regarding GSoC project, this is off topics here, please follow the relevant instructions on GeomScale wiki for GSoC23.

Soumya624 commented 1 year ago

Hi @vissarion , Double checked the version. And it seems they are same.

vissarion commented 1 year ago

Thanks, please open a PR.

Soumya624 commented 1 year ago

@vissarion ,

Please have a look at the PR(#275 )

Thanks!