search

DerwenAI / kglab

Graph Data Science: an abstraction layer in Python for building knowledge graphs, integrated with popular graph libraries – atop Pandas, NetworkX, RAPIDS, RDFlib, pySHACL, PyVis, morph-kgc, pslpython, pyarrow, etc.
https://derwen.ai/docs/kgl/
MIT License
574 stars 65 forks source link

Improve installation #272

Open Mec-iS opened 2 years ago

Mec-iS commented 2 years ago

I'm submitting a

Current Behaviour:

Current installation is possible only with pip

Expected Behaviour:

It may be good for accessibility to provide installation via python-poetry and suggest Anaconda in the README for people that prefer conda environments

cc: @ceteri

ceteri commented 2 years ago

Outstanding, using poetry would be a nice improvement for kglab !

We have support for conda (especially for the RAPIDS integration, e.g., cuGraph, etc.) and there's a brief mention in the README for that.

ghost commented 2 years ago

@ceteri @Mec-iS are you wanting to move the KGLab package build to poetry?

Mec-iS commented 2 years ago

we would like to provide different alternatives for installation, one of which can be poetry. ideally the requirements definition should be same between requirements.txt and pyproject.toml

ceteri commented 2 years ago

Something less 2008-ish :)

It gets interesting to support a wider range of alternatives for installation, given that the RAPIDS parts can only be installed with conda which is specifically configured through the NVIDIA release selector https://rapids.ai/start.html#get-rapids

That said, we don't want to become a conda-only package.

ghost commented 2 years ago

I'll do a bit of poking around and come back with some ideas for pipenv and poetry

ghost commented 2 years ago

I will update the readme with install instructions for poetry and pipenv to install as a pypi package to use in a project. I will also make updates to install the dev/test tutorial requirements with poetry and pipenv. I'll create a pyproject.toml for poetry and a Pipfile for pipenv.

Created https://github.com/DerwenAI/kglab/issues/275 to split changes

jake-aft commented 1 year ago

This is the related PR for this issue https://github.com/DerwenAI/kglab/pull/299