Create dingo-walk PyPi package

[vfisikop]

This is needed for creating a pip package (since the dingo name is already used).

[hariszaf]

Hi @vfisikop !
Having dingo on PyPi would be great 🎉 and dingo_walk is just fine.
However, I do not think that we need to change the namespace all over the project.
It's possible to have something like

pip install dingo_walk

and then

import dingo
..

If there is not a reason I am not aware of to change the namespace all over, I would suggest to go with the above solution.

[vfisikop]

I proposed that change since I think it is more clear to have a unique name everywhere (namespace, package etc). Also since there is a package named dingo already (that has also the same name for its namespace) there is a conflict.

[hariszaf]

I have no strong opinion on this, I just really enjoyed the sklearn story when I first heard of it. 👨‍💻

[vfisikop]

@TolisChal what is your opinion?

[vfisikop]

I agree to keep the dingo name everywhere instead of PyPi package where the name would be dingo-walk. It is clear and we do not have to use the dingo_walk (with "_") since in any case we cannot do import dingo-walk.

I updated my PR.

[hariszaf]

Thanks @vfisikop for the PR!

Going through that it made me think we could deploy something this so we automatically fix formatting.

Not sure if that is the best example though; it's the first thing close to what I was thinking. 😃

[hariszaf]

funny we had numpy twice.

[hariszaf]

I have never tried this before, but since the PyPi is the main goal of this PR, do you think we should consider setting something like this ?

[vfisikop]

That could be an option, we should definitely discuss it.

[vfisikop]

However, the process of publishing with poetry is quite straightforward. For example, I am following this guide https://www.digitalocean.com/community/tutorials/how-to-publish-python-packages-to-pypi-using-poetry-on-ubuntu-22-04
So, I would not add more complication at this stage.

[vfisikop]

Thanks @vfisikop for the PR!

Going through that it made me think we could deploy something this so we automatically fix formatting.

Not sure if that is the best example though; it's the first thing close to what I was thinking. 😃

You mean for this PR or in general? We can discuss the auto-formatting. I have experience with that in Python but I have mixed filings in general. @TolisChal had used black for dingo at some point, AFAIKT.

[hariszaf]

In order to make PyPi work in my case, I had to do something like:

CFLAGS="-I/usr/include/suitesparse" pip install dingo-walk