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

Using repo as dependency results in partial install #105

Open
jnv opened this issue Sep 27, 2023 · 0 comments
Open

Using repo as dependency results in partial install #105

jnv opened this issue Sep 27, 2023 · 0 comments

Comments

@jnv
Copy link

jnv commented Sep 27, 2023

Hi, I'd like to use this project as a dependency via Poetry, however, when I install the package from git (since the version on PyPI is vastly outdated), it seems all the submodules are missing.

Reproduction steps:

  1. Create a new folder
  2. Setup new Poetry project: poetry init
  3. Add this repo as dependency: poetry add git+https://github.com/aiwaves-cn/agents.git
  4. Check the site-packages/agents and observe that only top-level Python files are included; in my case:
.
├── evolve.py
├── __init__.py
├── SOP.py
├── State.py
├── template.py
└── utils.py

I suspect this could be regression introduced in 2ae5457. I'm no distutils expert, but it seems specifying just packages = ['agents'] will omit all subpackages (e.g. agents.LLM).

Perhaps one option to fix it would be to combine previously removed find_packages with package_dir like this?

packages=find_packages(where='src'),
package_dir={"": "src"}
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

1 participant