search

MDAnalysis / mdanalysis

MDAnalysis is a Python library to analyze molecular dynamics simulations.
https://mdanalysis.org
Other
1.26k stars 641 forks source link

remove analysis.psa in 3.0 #4542

Open orbeckst opened 3 months ago

orbeckst commented 3 months ago

Remove analysis.psa

IAlibay commented 3 months ago

Apologies if I don't understand, @orbeckst, don't we already have another issue for this?

As in, I assigned myself to complete that other issue.

IAlibay commented 3 months ago

Also you've opened a few of these @orbeckst, may I ask that we don't push for these to be done until we get closer to 3.0. i.e. let's not add these to the 3.0 merge pending queue

Just for CI alone the merge conflict is going to be more work than the act of deleting the code.

orbeckst commented 3 months ago

That's fair – It wasn't my intention to put them on an immediate TODO list. We should just do these PRs when we know that the next release is 3.0.

However, it's my understanding that all of this work falls under EOSS4 and is relevant to MDA 3.0 so I think they should be listed under these Projects, or is this not the case?

IAlibay commented 3 months ago

However, it's my understanding that all of this work falls under EOSS4 and is relevant to MDA 3.0 so I think they should be listed under these Projects, or is this not the case?

Sorry I think I don't understand what you mean. I probably completely missed what you're doing here apologies. We can put it on the agenda for the next call?

orbeckst commented 3 months ago

Sure, maybe I just need a refresher on how we use Projects in the issue tracker.