metadata icon indicating copy to clipboard operation
metadata copied to clipboard

[UI] Add lineage sections to Model Detail views

Open prodonjs opened this issue 6 years ago • 7 comments

/kind feature /area front-end

Retrieve and display related Data Set and Evaluation metrics for Models.

Mocks

prodonjs avatar Jul 01 '19 15:07 prodonjs

@zhenghuiwang Can you point me to any guidance or documentation on how we might be able to use the new events API methods you added in #53?

For example, using the data in https://kubeflow-v-n00.endpoints.zhenghui-kubeflow-2.cloud.goog: /api/v1alpha1/artifact_types/kubeflow.org/alpha/model/artifacts/40 is the model in question.

/api/v1alpha1/events/artifacts/40 gives back

{
   "events":[
      {
         "artifact_id":"40",
         "execution_id":"12",
         "type":"OUTPUT",
         "milliseconds_since_epoch":"1562114383006"
      }
   ]
}

Then, using the execution_id from the returned event, /api/v1alpha1/events/executions/12 returns

{
   "events":[
      {
         "artifact_id":"39",
         "execution_id":"12",
         "type":"INPUT",
         "milliseconds_since_epoch":"1562114382971"
      },
      {
         "artifact_id":"40",
         "execution_id":"12",
         "type":"OUTPUT",
         "milliseconds_since_epoch":"1562114383006"
      },
      {
         "artifact_id":"41",
         "execution_id":"12",
         "type":"OUTPUT",
         "milliseconds_since_epoch":"1562114383042"
      }
   ]
}

Would we then need to go about determining the type of each artifact listed there and retrieving them individually or is there something I am missing?

prodonjs avatar Jul 03 '19 19:07 prodonjs

You're right. It is a problem: the response doesn't have type info about the artifacts&executions hence you can't retrieve each individual artifact&execution.

Proposed fix: I will add response field artifacts: map from id to artifact, and field executions similarly.

zhenghuiwang avatar Jul 03 '19 21:07 zhenghuiwang

blocked by #84

prodonjs avatar Jul 08 '19 16:07 prodonjs

/reopen

PR#84 is merged and I've updated our dev KF cluster to master. You should see the artifact&execution map in the ListEvent response now.

zhenghuiwang avatar Jul 08 '19 22:07 zhenghuiwang

@zhenghuiwang: Reopened this issue.

In response to this:

/reopen

PR#84 is merged and I've updated our dev KF cluster to master. You should see the artifact&execution map in the ListEvent response now.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Jul 08 '19 22:07 k8s-ci-robot

/kind feature /priority p2

jtfogarty avatar Jan 14 '20 22:01 jtfogarty

@jtfogarty: The label(s) area/feature cannot be applied, because the repository doesn't have them

In response to this:

/area feature /priority p2

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Jan 14 '20 22:01 k8s-ci-robot