Skip to content

Comments

Improve development setup instructions, fix dependency list, and clarify documentation#1660

Open
VHemanth45 wants to merge 1 commit intoopenml:mainfrom
VHemanth45:Fix/Docs
Open

Improve development setup instructions, fix dependency list, and clarify documentation#1660
VHemanth45 wants to merge 1 commit intoopenml:mainfrom
VHemanth45:Fix/Docs

Conversation

@VHemanth45
Copy link

Metadata

Details

  • This PR updates the documentation and dependency instructions for the OpenML Python connector.
    The changes include:
  • Fixing typos and formatting in CONTRIBUTING.md for clarity and consistency.
  • Updating the recommended Python version for the development environment to 3.11 in the setup instructions.
  • Correcting the uv pip install command to use quotes for extras (uv pip install -e ".[test]").
  • Adding missing dependencies (openml, minio) to the [project.optional-dependencies] test section in pyproject.toml.
    Removing duplicate or unnecessary dependencies (such as openml-sklearn) from the test dependencies.
  • Minor whitespace and formatting fixes for better readability.
    These changes are necessary to ensure contributors have up-to-date and accurate instructions for setting up their development environment and running tests, reducing confusion and setup errors.
    No new functionality or tests are added in this PR; it is focused on improving the developer experience and documentation accuracy.

Copy link
Collaborator

@fkiraly fkiraly left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

Removing the openml-sklearn and adding openml as a dep changes behaviour. Can you explain why you are doing this in a documentation PR?

@VHemanth45
Copy link
Author

VHemanth45 commented Feb 20, 2026

Thank you for your feedback!
I initially set out to improve the documentation, but while setting up the environment, I noticed some discrepancies and dependency issues. To ensure a smoother setup for future contributors, I fixed these issues in the same PR.
You can refer this #1659

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Installation guide uses Python 3.8, which is incompatible with current OpenML requirements

2 participants