Remove torchvision and torchtext

[samet-akcay]

Description

• Remove torchvision and torchtext from the base requirements.

Changes

• Bug fix (non-breaking change which fixes an issue)
• Refactor (non-breaking change which refactors the code base)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

Checklist

• My code follows the pre-commit style and check guidelines of this project.
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing tests pass locally with my changes
• I have added a summary of my changes to the CHANGELOG (not for minor changes, docs and tests).

[codecov]

Codecov Report

Patch coverage: 94.73% and no project coverage change

Comparison is base (1407cfc) 86.77% compared to head (51a7a82) 86.77%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #903   +/-   ##
=======================================
  Coverage   86.77%   86.77%           
=======================================
  Files         165      165           
  Lines        6319     6322    +3     
=======================================
+ Hits         5483     5486    +3     
  Misses        836      836           

Impacted Files	Coverage Δ
...ib/models/components/feature_extractors/torchfx.py	92.98% <94.73%> (+0.38%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[djdameln]

Is this safe? How do we know the correct version is installed?

[ashwinvaidya17]

Is torchtext automatically installed? If I remember correctly, SC docker files contain additional torchtext dependency in the base image. But that maybe just for fixing the version.

[samet-akcay]

nstalled? If I remember correctly, SC docker files contain additi

We don't have a check for torch, but do have for these. Actually this would be even more problematic

[ashwinvaidya17]

nstalled? If I remember correctly, SC docker files contain additi

We don't have a check for torch, but do have for these. Actually this would be even more problematic

Alright. Let's roll with this then

[samet-akcay]

@ashwinvaidya17, I also feel that we should also have anomalib install option that comes with the CLI. I think that would be the safest bet