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

Docs/pipx #29

Merged
merged 6 commits into from
Jul 3, 2024
Merged

Docs/pipx #29

merged 6 commits into from
Jul 3, 2024

Conversation

Joao-L-S-Almeida
Copy link
Member

Pipx as an alternative to pip.

Signed-off-by: João Lucas de Sousa Almeida <[email protected]>
Signed-off-by: João Lucas de Sousa Almeida <[email protected]>
Signed-off-by: João Lucas de Sousa Almeida <[email protected]>
Signed-off-by: Joao Lucas de Sousa Almeida <[email protected]>
Copy link
Contributor

@CarlosGomes98 CarlosGomes98 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall, I'm not entirely convinced of the need for pipx. From my understanding, this is something usually used for commands that the user wants available from all their environments to be used in general. e.g. something like poetry.

Terratorch I believe is more of a dependency of a specific project. I can imagine, for instance, the case down the line where some projects require a certain version of terratorch, while others require a different version. In this case for a single user on his machine pipx would make life somewhat hard.

But I'm open to the fact I might be missing some good arguments for why its a good idea. Does it offer some advantage over pip for our case?

@CarlosGomes98 CarlosGomes98 mentioned this pull request Jul 2, 2024
@Joao-L-S-Almeida
Copy link
Member Author

Pipx installs a Python package in a similar way APT does for Debian-based Linux and Brew for Mac, it means, it installs the dependencies in background and creates an executable or a symlink shared on PATH, which allows the user run it without needing to create an environment. I think it could be useful for an user interested in just running fine-tuning experiments without needing to configure anything.

@CarlosGomes98
Copy link
Contributor

Ok, in the end it is one more alternative some users may prefer

Copy link
Contributor

@CarlosGomes98 CarlosGomes98 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After the change in tests/test_backbones.py is done will approve, LGTM

@CarlosGomes98 CarlosGomes98 merged commit 9d3da72 into main Jul 3, 2024
3 checks passed
@CarlosGomes98 CarlosGomes98 deleted the docs/pipx branch October 2, 2024 08:32
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

Successfully merging this pull request may close these issues.

2 participants