Deprecate ModelCardGenerator component

[codesue]

What does this pull request do?

• Removes the ModelCardGenerator component and the tfx dependency
• Updates documentation and adds a notice that the component is moving to tfx-addons
• Bumps the upper limits of tf-related dependencies so that model-card-toolkit will be compatible with newer versions of tfx

This change will be part of version 2.0.0.

Fixes #247, #202, #185, #93

How did you test this change?

Installed with pip install -e ".[test]". Ran all tests with find . -name '*_test.py' | xargs -I {} python {}.
See this gist for pip freeze and test output.

Note: The test added in this PR looks like it takes a long time to run, but the expensive part of that is importing model_card_tookit the first time. Running time on model_card_toolkit/__init__.py takes 7.27s user 1.55s system 90% cpu 9.727 total compared to 0.06s user 0.03s system 55% cpu 0.156 total when running the code for the added test on its own.

How did you document this change?

• Updated guides and READMEs.
• Added a notice as a NotImplementedError that's raised when importing from the removed module.

Before submitting

Before submitting a pull request, please be sure to do the following:

• Read the How to Contribute guide if this is your first contribution.
• Open an issue or discussion topic to discuss this change.
• Write new tests if applicable.
• Update documentation if applicable.

Co-authored-by: Suzen Fylke codesue@users.noreply.github.com
Co-authored-by: Hannes Hapke hanneshapke@users.noreply.github.com

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[codesue]

Hi @casassg and @ahmed-bd, would you please review this PR when you get a chance. It's a larger and breaking change, so I'd appreciate more feedback or approvals. 🙏

[casassg]

overall lgtm, an idea here may be to try to import the component from tfx-addons:

try:
  from tfx-addons.model_card_generator import ModelCardGenerator
except ImportError:
  ModelCardGenerator = None

This way users can optionally install tfx-addons and still continue using the component without breakage. it's a bit complex to be fair, so not 100% sure about it, but may be worth a shot

[codesue]

Apparently this branch was set up to automatically dismiss PR approvals when new commits are pushed. Just changed it to not do that. 😅

[hanneshapke]

@codesue Do you mind keeping the dismissed stale reviews setting? It helps with tracking changes and avoids last-minute changes.

[codesue]

@codesue Do you mind keeping the dismissed stale reviews setting? It helps with tracking changes and avoids last-minute changes.

@hanneshapke I changed it back.