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

Dependency Management #9

Closed
paketb0te opened this issue Jan 23, 2023 · 8 comments
Closed

Dependency Management #9

paketb0te opened this issue Jan 23, 2023 · 8 comments
Assignees
Labels
enhancement New feature or request

Comments

@paketb0te
Copy link
Contributor

Once again, this is a great tool!

Can I interest you in using poetry for packaging and dependency management?

I had pretty good experiences with it, can definitely recommend.

If you prefer a simpler approach like requirements.txt that's fine too (but I really recommend to take a look at poetry 😆)

(Would look something like THAT)

@initialcommit-io
Copy link
Contributor

Hi and thank you! Thx for reaching out.

Right now I'm just using entries in the setup.py like:

install_requires=[
    'gitpython',
    'manim',
    'opencv-python-headless',
],

Which takes care of collecting everything automatically. It looks like the poetry.lock file you generated breaks down every dependency and stores it. Is the benefit of this that it more or less guarantees that the build will succeed by defining specific versions of every dependency? Seems a bit messier to need to store all of this info though when most of the build dependences are nested within each other. Like, most of those dependencies come from Manim...

@paketb0te
Copy link
Contributor Author

yeah, you only need to specify them explicitly in the pyproject.toml file (with constrints, if you wish, like "needs at least verion X.Y of package Z").

The file poetry.lock is created automatically by poetry (poetry lock) to pin all dependencies to fixed versions, which as you mention helps ensure that the build is reproducible / identical for everybody.

@initialcommit-io
Copy link
Contributor

I see, thanks for the suggestion. I need to learn more about poetry and other options and will get back to you on this one.

@initialcommit-io
Copy link
Contributor

@paketb0te I'm playing around with poetry and got a similar pyproject.toml file to the one at your link.

Just a few questions about some extra info that is currently in the setup.py file which I wasn't prompted for by the interactive poetry setup:

-url
-keywords
-project_urls (homepage and source)
-entry_points

Do you know how these can be specified with poetry?

@paketb0te
Copy link
Contributor Author

paketb0te commented Jan 27, 2023

According to the docs, we can just specify those in pyproject.toml, e.g. repository = "https://github.com/initialcommit-com/git-sim"

So it would look something like this:

[tool.poetry]
name = "git-sim"
version = "0.1.2"
description = "Simulate Git commands ..."
authors = ["Jacob Stopak <jacob@initialcommit.io>"]
license = "MIT"
readme = "README.md"
packages = [{include = "git_sim"}]
homepage = "https://git-sim.org/"
keywords = ["git", "sim", "simulation", "..."]
repository = "https://github.com/initialcommit-com/git-sim
...

If I understand correctly, you can define the equivalent to entry_points via plugins in a subsection in the pyproject.toml (see docs)

edit: Apparently you can also define arbitrary URLs besides homepage and repo: urls

@buhtz
Copy link

buhtz commented May 9, 2023

In relation to #85

I'm not well experienced packaging big projects but small ones.
In #85 you also stated that you are not well experienced with packaing. Because of that I would like to give an advice to save your resources, your time, your nerves and keep the project (and its maintainer) healty. 😄

  1. Use pyproject.toml (without setup.py/setup.cfg) and nothing else.
  2. Use setup.py/setup.cfg only if you really know why you need it. There are reasons but most of them time that are special reasons.
  3. Don't use poetry, tox or something like this. This are very complex big tools for big purposes. Use them only if you really have a hard indication for it. Otherwise it would waste a lot of your time.

Again about 2. and 3. This are not bad tools. The problem is just that they are recommended in every blog post etc. But they are intended to solve problems that "small projects" usually don't have. Keep your solutions grounded.

If you like you can contact me and I explain you my real world projects how I solved somethings just using pyproject.toml not using anything else.

@paketb0te
Copy link
Contributor Author

Hi @buhtz, thanks for your input on this.

I was not aware that you can also specify dependencies in pyproject.toml, that's super neat!

-> That kinda obsoletes the usage of poetry for now, I guess.

@initialcommit-io let's close this issue.
@buhtz I'd be glad to work on #85 together, if you are up for it. I have to do some reading about project layout before though 😁

@initialcommit-io
Copy link
Contributor

Sounds good @paketb0te, closing this one for now so pyproject.toml work can be continued thru #85 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants