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.

Sign up for GitHub

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

Jump to bottom

Use poetry as the main installation method for vctl install #196

Closed
craig8 opened this issue Jul 31, 2024 · 2 comments
Closed

Use poetry as the main installation method for vctl install #196

craig8 opened this issue Jul 31, 2024 · 2 comments
Assignees
@schandrika

Comments

@craig8
Copy link
Contributor

craig8 commented Jul 31, 2024

Is your feature request related to a problem? Please describe.
Pip is horrible for dependency management. Our solution is to use poetry with a lock file for the virtual environment to manage the dependencies. This is especially important for developers. When installing a listener agent will break the server loading of some items. This we believe will be alleviated with the addition of poetry.

Describe the solution you'd like
We want to have poetry to be used for the addition of agents to the platform rather than pip.
schandrika added a commit to schandrika/volttron-core that referenced this issue Aug 7, 2024
@schandrika
changed vctl install to use poetry add instead of pip install. Added …
8b52a46 
…--dev option to volttron start to use poetry correctly. Fix for issue eclipse-volttron#196
@schandrika schandrika mentioned this issue Aug 7, 2024
Merged
craig8 added a commit to craig8/volttron-core that referenced this issue Aug 7, 2024
@craig8
Merge pull request #8 from schandrika/poetry_install_changes
94b2237 
Fix for issue eclipse-volttron#196. vctl should use poetry instead of pip to install agents
schandrika added a commit to schandrika/volttron-core that referenced this issue Aug 9, 2024
@schandrika
Fix for issue eclipse-volttron#196-change vctl install to use poetry.…
55b1932 
… implementation of vctl install --force
craig8 pushed a commit to craig8/volttron-core that referenced this issue Aug 9, 2024
@schandrika @craig8
Fix for issue eclipse-volttron#196-change vctl install to use poetry.…
d8b032f 
… implementation of vctl install --force
@craig8
Copy link
Contributor Author

craig8 commented Sep 30, 2024

@schandrika I believe we will be done with this as soon as the --dev is done correct?

@schandrika schandrika mentioned this issue Sep 30, 2024
Merged
@schandrika
Copy link
Contributor

Fixed in version 2.0.0rc0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@schandrika schandrika

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@craig8 @schandrika