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

Pydantic-settings-yaml is only on pypi, not on git or conda-forge #25

Closed
claudiushaag opened this issue Jan 5, 2025 · 5 comments
Closed

Comments

@claudiushaag
Copy link

claudiushaag commented Jan 5, 2025

I was trying to package bagels on conda-forge, as installing cli-tools via pixi global is very convenient.

But pydantic-settings-yaml is a obscure dependency. Also see this SO-post, which supports that confusion.

As the package is not documented well, I would opt to rather not add it to conda-forge in order to enable the build of bagels. Would you be open to circumvent its usage? There is also the package pydantic-yaml, or yaml-settings-pydantic, which seem to be more actively developed, which might provide a similar functionality, without having a look at the source code.

I would appreciate a reply!

@EnhancedJax
Copy link
Owner

Hi, thanks for the issue! I'm not sure if I understand what conda-forge is, and how pydantic-settings-yaml is affecting the packaging process. I do understand the risk of using an obscure dependency though, so I'll look into the packages you've suggested! Feel free to create a PR!

@claudiushaag
Copy link
Author

Conda-Forge is a community hosted channel for conda packages. These packages enable the installation of binaries and are quite popular in the scientific world. In particular, the package manager pixi enables the super easy installation of a CLI-Tool, similar to uv tools (just with potentially a wider spectrum of packages). I think, bagels would also benefit from being installable via conda.

But, to be installable, all the dependencies must also be on conda-forge. pydantic-settings-yaml is not on conda-forge, and it's not really feasible to put it there. Therefore, if a conda installation is interesting to you, pydantic-settings-yaml would need to be circumvented (I think this affects mainly the Config class).

@EnhancedJax
Copy link
Owner

I see, thanks for the detailed explanation!

@EnhancedJax
Copy link
Owner

I've removed these dependencies in the latest version, please see if you can package bagels on conda-forge!

@claudiushaag
Copy link
Author

Thank you, that looks good from my side! I put in a PR (the usual procedure to create a conda package), where a comment of you would be needed to register you as a maintainer (anything will work, like "Willing to be listed as a maintainer"), so you have control over updates in the feature, even though I will also be maintaining.
There are some things to be sorted out from conda side, and I will have a look into that, but as soon as that is finished, bagels should be ready!

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

No branches or pull requests

2 participants