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

Unnecessary and confusing nesting of poetry projects #74

Closed
jonathanunderwood opened this issue Sep 13, 2023 · 2 comments
Closed

Unnecessary and confusing nesting of poetry projects #74

jonathanunderwood opened this issue Sep 13, 2023 · 2 comments

Comments

@jonathanunderwood
Copy link

The instructions given in the README mean that you end up creating twopoetry virtual environments, one of which is created from running poetry inside the first poetry virtual environment. This appears to be done under the assumption that the user wants to have a number of linkml projects underneath a single linkml projects folder.

Another assumption is that the user will run Makefile commands with a pre-activated poetry shell (i.e. poetry shell). I don't think this is how most users use poetry - observing how people use poetry, they tend to not activate the shell and then run commands, but instead use poetry run ....

My recommendation is that this set of instructions is redesigned to focus on the simpler use case of "I want to create a single linkml project in a folder, and I want to run make without first activating a poetry shell". The latter part would be achieved by having the Makefile commands prefixed with poetry run as appropriate.

@dalito
Copy link
Member

dalito commented Nov 24, 2023

This nesting is not "unnecessary" but required to isolate the environments for the installation tooling from the created linkml-project. But I agree that it is a bit confusing.

If presence of pipx could be assumed, the instructions in Step 1 & 2 would reduce to pipx install poetry, pipx install cruft. This would be much simpler and avoid the confusion.

In my opinion it is good a advice to have pipx on the system when you work with Python but am not sure if I am part of a minority. pipx has worked great for me since years on Windows & Linux.

@pkalita-lbl
Copy link
Contributor

With #84 the instructions have been rewritten to advise using pipx and not have the nested Poetry projects.

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

No branches or pull requests

4 participants