Code Monkey home page Code Monkey logo

Comments (9)

RSchwan avatar RSchwan commented on June 11, 2024

Wow, that's amazing, thanks for all the effort.

In fact, I have also been working on an R interface myself (see the repo here). It's based on osqp-r, my limited knowledge of R, and some ideas from the clarabel-r package. I also sent it to an active R user for testing, before trying my luck with CRAN myself. Hence, the reason why I haven't advertised it yet.

Looking through your repo, I can see a lot of similarities, but also details and design decisions which are probably superior to what I threw together. Especially, considering your expertise in R. Hence, maybe we could take the best from both repos and combine them in a single package. For example, in your interface, the API for updating the data and thus reusing the ordering and factorization is missing. On the other side, your vignettes documentation is a nice addition.

from piqp.

bnaras avatar bnaras commented on June 11, 2024

Thanks. Yes, I agree that the updating etc. are good to add, but this is all I did in between my day job on Friday. I have also cleaned up some tests and will make them comprehensive.

I am happy to take the best and merge them together. Also happy to move this under your Github org, just as I did for clarabel-r which I also authored. (I also maintain osqp-r!)

I should mention my interest is in making these solvers available to our package CVXR, the R cousin of cvxpy...

from piqp.

bnaras avatar bnaras commented on June 11, 2024

The merge is now complete and the package is CRAN-ready as the README badge shows. One Q I have is how strict we should be about C++14. This throws a note in CRAN checks if we insist on it (surmountable with comments to CRAN) but if we need to, why not insist on C++17?

from piqp.

RSchwan avatar RSchwan commented on June 11, 2024

C++17 also works, the minimum requirement is just C++14. I thought that it's better to specify the minimum requirement (in this case C++14), but if C++17 gets rid of the warning and makes it easier for CRAN we can also go with C++17.

from piqp.

bnaras avatar bnaras commented on June 11, 2024

Ok, I'll leave it as C++14 for correctness.

One last question. I think the package web page and repo more aptly belongs under your github org than mine as Clarabel-r does. Can we move it there? I can continue to maintain it so that would not be a problem.

from piqp.

RSchwan avatar RSchwan commented on June 11, 2024

Sure, this would be great. In this case, I'll remove/move the repo on our end, so we can move/transfer your repo there. Or should I just push your code (with history) into a new repo in our org (after removing the old one), and add you as a contributor? Not sure what's easier.

from piqp.

bnaras avatar bnaras commented on June 11, 2024

As long as I can commit, either one is fine, per your convenience.

from piqp.

bnaras avatar bnaras commented on June 11, 2024

It's now on CRAN, so all done.

from piqp.

RSchwan avatar RSchwan commented on June 11, 2024

Awesome, thank you so much for all the effort.

from piqp.

Related Issues (6)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.