mdanalysis icon indicating copy to clipboard operation
mdanalysis copied to clipboard

Make matplotlib an optional dependency

Open IAlibay opened this issue 2 years ago • 4 comments

In the spirit of reducing the footprint of MDAnalysis, we should make matplotlib an optional dependency.

This currently depends on #4312, and the HOLE2 removal - once those are done we can do this.

IAlibay avatar Nov 05 '23 09:11 IAlibay

From last time I looked for 3.0 roadmap there are a few other spots.

  • analysis/dihedrals.py (quite a few)
  • analysis/polymer.py
  • analysis/psa.py

How do we want to approach? We could deprecate and shift these methods to examples?

hmacdope avatar Nov 05 '23 21:11 hmacdope

PSA, HOLE2, and streamlines is probably going away.

Polymer & dihedrals are sticking around as far as I'm aware - personally I'd just wait until we make the first 3 MDAKits and then make the methods from the latter two optional - it's mostly plot calls, so you can separate them out without hurting functionality.

IAlibay avatar Nov 06 '23 08:11 IAlibay

Hey, I have looked over the codebase and I would like to work on this.

SawanSharma996 avatar Jan 24 '24 18:01 SawanSharma996

@SawanSharma996 unfortunately this change is being held up by other projects (hole2 and streamlines migrations) and isn't available to work on right now.

IAlibay avatar Jan 24 '24 18:01 IAlibay