augur icon indicating copy to clipboard operation
augur copied to clipboard

remove unused dependencies

Open MoralCode opened this issue 2 months ago • 4 comments

Description This change:

  1. removes several dependencies that were seemingly never used
  2. relocates a couple of dependencies to either the devdependencies area, or removes them since they are transitive dependencies

This PR is a major step towards resolving #3334 (and in future, updating dependencies)

Notes for Reviewers

Signed commits

  • [x] Yes, I signed my commits.

I should probably squash these commits

MoralCode avatar Oct 23 '25 19:10 MoralCode

None of them should be used, period. For basically every single one of these, I ctrl-F-ed after the name, and looked to see if we were importing it anywhere throughout the entire codebase.

I did also have the idea of making it an optional sub package too, but I'm not sure how that works

MoralCode avatar Oct 24 '25 22:10 MoralCode

some of these are for sure still being used by the ml workers

"scikit-image==0.19.1", "scikit-learn==1.5.0", sklearn-crfsuite

h5py is  imported by keras (transitive) so can prob be left out

MoralCode avatar Oct 29 '25 00:10 MoralCode

needs rebase

MoralCode avatar Nov 19 '25 00:11 MoralCode

Rerunning failed end to end test.

sgoggins avatar Dec 09 '25 21:12 sgoggins