[MNT] Remove `__version__.py` module and define `__version__` in `__init__`

[geetu040]

Currently, the package defines __version__ inside a __version__.py module and then re-exports it from openml/__init__.py.

This results in two objects named __version__:

• the module openml.__version__
• the variable __version__ imported from that module

This kind of name masking between a module and a variable is generally discouraged in Python, as it can lead to confusion and unclear import semantics.

Todo:

• Remove openml/__version__.py
• Define __version__ directly inside openml/__init__.py
• Update any internal imports accordingly

This keeps a single, conventional openml.__version__ attribute while avoiding module/variable name collisions and simplifying the package structure.

Initial Source/Discussion: aiondemand/aiondemand#70

[amanmurari]

can I work on this Issue

[geetu040]

can I work on this Issue

This has already been taken. I can see a 2 corresponding PRs.

You can may be take a look at other issues, or review these PRs with your suggestions.
In-case you are looking for good-first-issues, you can look find them here: Good First Issues
Here are some more umbrella issues: #1649, #1633

[geetu040]

There are two PRs open for the same issue:

• @omkar-334’s [MNT] Define __version__ in __init__.py #1653
• @moksha-hub’s [MNT] Remove __version__.py, define __version__ in __init__.py #1661

Before opening a PR, please leave a quick comment on the issue to indicate you're working on it. That helps avoid duplicated effort and makes it clear who should be credited for the fix.

[moksha-hub]

@geetu040 Thanks i will drop and close my pr.
Thank you

[Abhishek9639]

@geetu040,
I would love to work on this issue. Could you please assign it to me?

[omkar-334]

@Abhishek9639 please read the above comments. this issue already has a PR.

Before opening a PR, please leave a quick comment on the issue to indicate you're working on it. That helps avoid duplicated effort and makes it clear who should be credited for the fix.

See this

[Abhishek9639]

Sorry, I didn’t notice that this issue already has a PR. I will look for another issue to work on.