Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compatibility with 1.1 #1

Open
patrick-nicodemus opened this issue May 6, 2024 · 8 comments
Open

Compatibility with 1.1 #1

patrick-nicodemus opened this issue May 6, 2024 · 8 comments

Comments

@patrick-nicodemus
Copy link

Is there any reason this should be incompatible with owl 1.1?

If it is compatible, the official opam file should be updated.

There are currently no tests.

@patrick-nicodemus
Copy link
Author

Ok. I compiled it locally and tested it with some of the examples from the tutorial page and it seems to work.

@jzstark jzstark closed this as completed May 7, 2024
@patrick-nicodemus
Copy link
Author

This issue is not closed. I said I tested it. The issue is to update the official opam repository here is up to date.
https://github.com/ocaml/opam-repository/tree/master/packages/owl-plplot

I think the best solution is to make an official new release of owl-plplot (on this git repo) and then ask the opam maintainers to accept an updated opam file in their repository.

@patrick-nicodemus
Copy link
Author

@jzstark
Can you please reopen the issue or if possible give me permissions for this repository?

@patrick-nicodemus
Copy link
Author

I am not an experienced ocaml developer but this page seems out of date.
https://opam.ocaml.org/packages/owl-plplot/

The source URL for the latest version is not appropriately decoupled/redirected to this current repository, note that it says https://github.com/owlbarn/owl/releases/download/1.0.2/owl-1.0.2.tbz
The owl version required is either 1.0.2 or 1.0.2-1.

My understanding is that when a user writes "opam install owl-plplot", this file controls the installation.

https://github.com/ocaml/opam-repository/blob/a799daa56b1b3824d067c7cc92be627eef88aafe/packages/owl-plplot/owl-plplot.1.0.2/opam

@jzstark jzstark reopened this May 23, 2024
@mseri
Copy link
Member

mseri commented May 23, 2024

@patrick-nicodemus you are right. The best idea would be to make a release depending on owl >= 1.1 and pointing to a 1.1 release in this repo

@Ecsodikas
Copy link

Hello, did something happen in the meantime? I was trying to use the plotting library but the dependency locks me out of using ocaml 5.2.0.

Thanks for your hard work!

@patrick-nicodemus
Copy link
Author

I don't think there are updates so far, I think someone with permissions has to do it. If you clone the repo and change the minimum version number in the opam file you can install it locally.

Plplot itself is well documented and there is OCaml sample code available. It is a very imperative style, not super idiomatic, but super readable. The functions in owl plplot file are more convenience functions so you don't have to write your own histogram routine for plplot, not a fundamental coupling necessary to use plplot, you can use plplot directly without much hassle. I was pleasantly surprised at the elegant design of the library compared to Matplotlib.

@mseri
Copy link
Member

mseri commented Jul 12, 2024

I think the release has been made already enabling ocaml < 5.2. The problem is that owl itself is not compatible with ocaml 5.2 and it is nontrivial to make it work: owlbarn/owl#657

As @patrick-nicodemus mentions, if you don’t need owl you can still use the ocaml plplot package: https://ocaml.org/p/plplot/5.12.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants