search

owlbarn / owl

Owl - OCaml Scientific Computing @ https://ocaml.xyz
MIT License
1.2k stars 120 forks source link

Reintroduce owl-plplot #659

Closed mseri closed 3 months ago

mseri commented 4 months ago

owl-plplot provided a convenient and seamless way to plot, and is also used in the documentation. It is such thin layer and so easily installable that it is a pity that it has been removed.

jzstark commented 4 months ago

The plplot module itself is a thin layer, but it depends on the plplot library which is a bit heavy. The previous idea is to reduce external reliance such as eigen and plplot. Perhaps we can follow the numpy+matplotlib paradigm and put the plplot stuff in a separate repo in owlbarn.

mseri commented 4 months ago

I understand removing eigen since the way it needs to be compiled to work properly is simply a nightmare. Similarly I understand that removing aeos simplifies the code.

But ocaml-plplot is a rather stable and well designed package, and owl-plplot has always worked fine on all systems so far, so in principle you could keep it there at no extra cost.

in any case, I think the location does not matter. It is perfectly fine on a separate repository if you prefer it that way. Without owl-plplot there is no owl ergonomic plotting solution at the moment so it would be nice not to lose it

jzstark commented 3 months ago

True, I totally agree that plotting is an important part in numerical computing, just like NumPy is accompanied by Matplotlib. I've added it to the Todo list in the Project. It would be great if any one interested can help to add this package.

mseri commented 3 months ago

If you create an empty owl-plplot repository and give me (also just temporary) writing rights, I can push my fork (which is just a copy of the old sources) there

jzstark commented 3 months ago

Hi Marcello, I've created the repo and a temp write access as suggested. Thanks a lot for helping to set it up!

On Thu, 14 Mar 2024 at 17:35, Marcello Seri @.***> wrote:

If you create an empty owl-plplot repository and give me (also just temporary) writing rights, I can push my fork (which is just a copy of the old sources) there

— Reply to this email directly, view it on GitHub https://github.com/owlbarn/owl/issues/659#issuecomment-1997868022, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAK6ZA4O6P44PHSIDGY34QTYYHGWRAVCNFSM6AAAAABEJKOG2WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJXHA3DQMBSGI . You are receiving this because you commented.Message ID: @.***>

mseri commented 3 months ago

Done