Rocco Meli
Rocco Meli
@amkrajewski could you please briefly summarise the progress on the items listed here? Thank you.
Thanks @Henrium. @amkrajewski, I think we can close the issues then. @Henrium, would you mind looking at your checklist in https://github.com/openjournals/joss-reviews/issues/6731#issuecomment-2099348790 and see if there are still outstanding items to...
Hi @mattiafelice-palermo. This looks like a very useful analysis class! Thank you for the detailed explanation and information. It is definitely an addition we should consider. However, I should point...
BTW, parallelization can now be [handled by `AnalysisBase`](https://docs.mdanalysis.org/stable//documentation_pages/analysis/parallelization.html#). Since you used the second approach, it might be reasonably straightforward to refactor your class to use what MDAnalysis provides.
Hi @amkrajewski and @bdice is this issue still ongoing, or it can be resolved?
Thank you for the update! FYI, I'll be away for the next two weeks, apologies in advice if my replies will be slow.
Hi @bdice, when you evaluate the latest changes from @amkrajewski, please update your checklist in https://github.com/openjournals/joss-reviews/issues/6731#issuecomment-2183177055 accordingly, so that we can keep track of the progress. Many thanks!
@MDAnalysis/coredevs @marinegor The companion PR https://github.com/MDAnalysis/mdanalysis/pull/4717 add a `black` configuration to `package` and `testsuite` so that the commands in the documentation ``` black package black testsuite ``` do nothing.
Thanks @IAlibay. I'll merge this if/when https://github.com/MDAnalysis/mdanalysis/pull/4725 is merged.
> I have opened an issue on the feedstock, let's see what happens. Thanks @IAlibay! I'd be OK to be added as a maintainer if needed.