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

Expose tables under topology for 2.x #4753

Closed
IAlibay opened this issue Oct 20, 2024 · 2 comments
Closed

Expose tables under topology for 2.x #4753

IAlibay opened this issue Oct 20, 2024 · 2 comments

Comments

@IAlibay
Copy link
Member

IAlibay commented Oct 20, 2024

Related to #4748

Expected behavior

In a non-API breaking release, I would expect to find the things I used to use in the same place, unless it was a bug and/or a DeprecationWarning had been raised for a while.

Actual behavior

MDAnalysis.topology.tables no longer exists in the current develop.

Proposal

We add it back with a warning.

@IAlibay IAlibay added this to the Release 2.8.0 milestone Oct 20, 2024
@lilyminium
Copy link
Member

+1, sounds good to me, following the same logic as in #4751

@orbeckst
Copy link
Member

Can we also add a DeprecationWarning, given that it will go away and people seem to be using it directly?

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

No branches or pull requests

3 participants