added instructions to build local python package

[codecov[bot]]

Codecov Report

Merging #215 (2ecabcd) into master (eb8ab74) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #215   +/-   ##
=======================================
  Coverage   71.48%   71.48%           
=======================================
  Files         135      135           
  Lines        9114     9114           
=======================================
  Hits         6515     6515           
  Misses       2599     2599           

[Fratorhe]

Hi,

It can be much easier. Simply do: pip install .

If you get an error related to PEP517 update essential packages: pip install --upgrade pip setuptools wheel

Best, Fran

[rdbisme]

As @Fratorhe said. Closing! :)

pip install . is enough.

As I have a bit of spare time I'll enable the push of the pre-compiled wheel to PyPi for super easy install.

[rdbisme]

Actually documenting what @Fratorhe said would be beneficial. :)

[ggange]

Hello @rdbisme, here are the updated instructions (I added a small remark regarding zsh on macOS, since it was object of another question!). Thank you (@Fratorhe to you too)!

[rdbisme]

Closing as stale. Feel free to re-open addressing the comments from me and @Fratorhe.

[ggange]

Hi @rdbisme, is there something missing in the installation.md that is preventing this merge? I should have included yours and @Fratorhe comments in the instructions...

[rdbisme]

Hi @rdbisme, is there something missing in the installation.md that is preventing this merge? I should have included yours and @Fratorhe comments in the instructions...

The section ## Build Python package locally is not correct. pip install . is enough. The rest of the information is redundant with what is in pyproject.toml and already taken into account by pip install ..

So the information about installing the Python wrappers is just Run pip install . in the root of the repo.

[ggange]

ok, thanks! Like this? :)