feat: add tf1 metadata builder #526
Conversation
|
Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot. |
| ) | ||
| from google.cloud.aiplatform.explain.metadata import metadata_builder | ||
|
|
||
| try: |
There was a problem hiding this comment.
This import check should be done within methods.
|
|
||
| Args: | ||
| model_path: | ||
| Required. Path to load the saved model from. |
There was a problem hiding this comment.
Is this local, GCS, or both?
There was a problem hiding this comment.
Both, updated comments for both Tf1 and 2
| import tensorflow.compat.v1 as tf | ||
| except ImportError: | ||
| raise ImportError( | ||
| "Tensorflow is not installed and is required to load saved model. " |
There was a problem hiding this comment.
This should most likely be a more qualified statement as full will install TF 2. We may want to direct the user to install Tensorflow 1.15 directly here.
No description provided.