Skip to content

chore: update README.md of zenml-support-agent to correct installation path #201

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

Merged
merged 1 commit into from
Apr 30, 2025

Conversation

jadelapuente
Copy link
Contributor

Problem
Running the installation command from zenml-support-agent fails.

$ pip install -r src/requirements.txt
Defaulting to user installation because normal site-packages is not writeable
ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'src/requirements.txt'

This is because requirements is in the root directory and there is no src subdirectory.

Solution
Remove src/ from the path.

Alternative
Create src directory, but I chose not to because all other directories except 1 have this file structure.

Copy link

dagshub bot commented Apr 30, 2025

@strickvl strickvl requested review from Copilot and marwan37 April 30, 2025 11:54
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the README.md file in the zenml-support-agent repository to correct the installation path for the requirements file.

  • Removed the incorrect "src/" prefix from the pip install command.
  • Clarified the installation instructions to reflect the actual file structure.

@strickvl strickvl added the documentation Improvements or additions to documentation label Apr 30, 2025
@strickvl strickvl requested review from strickvl and removed request for marwan37 April 30, 2025 11:55
Copy link
Contributor

@strickvl strickvl left a comment

Choose a reason for hiding this comment

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

You're totally right! Thanks for the contribution!

@strickvl strickvl merged commit 2954142 into zenml-io:main Apr 30, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants