Coder Social home page Coder Social logo

frontend's Introduction

OpenSSF Best Practices OpenSSF Scorecard CLOMonitor

Kubeflow the cloud-native platform for machine learning operations - pipelines, training and deployment.


Documentation

Please refer to the official docs at kubeflow.org.

Working Groups

The Kubeflow community is organized into working groups (WGs) with associated repositories, that focus on specific pieces of the ML platform.

Quick Links

Get Involved

Please refer to the Community page.

frontend's People

Contributors

avdaredevil avatar bobgy avatar capri-xiyue avatar dushyanthsc avatar jlewi avatar kwasi avatar realsen avatar rui5i avatar zijianjoy avatar

Stargazers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar

Watchers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar

frontend's Issues

[MLMD] Add snapshot testing for LineageView

/area metadata
/priority p0

Background

59cbf3d introduced a regression where resource descriptions were missing. Even though the PR would have required updating snapshots, the missing data would have been very noticeable in the empty descriptions.

We should add tests that handles older data where fields are in ArtifactProperties as well as newer data where fields are in ArtifactCustomProperties.

See https://github.com/kubeflow/metadata/blob/master/frontend/src/pages/ArtifactDetails.test.tsx and its import of testModel for how to construct an instance of the GetArtifactById response with the correct test data.

[MLMD Lineage] Some Artifact and Execution names are null

/kind bug
/area front-end

Issues

  • The null titles are not shown in the ActionBar so the items become unclickable
  • Artifact and Execution cards display "null".

Screenshot: https://screenshot.googleplex.com/8PptgokgKXb

Repro Steps:

Note:

This may be partially a non-issue if the issue has been fixed in a later version
of the sample. The first investigation should be to try using a newer version:

https://github.com/kubeflow/pipelines/tree/master/samples/core/parameterized_tfx_oss

Either way, the issue of null titles showing up in the action bar as emtpy
spaces MUST be fixed.

[MLMD][BugBash] The execution -> artifact link in lineage graph is wrongly connected

/priority p0
/area front-end
/assign @kwasi

Description

It looks like downstream artifacts are connected to the first upstream execution only. See screenshot for sample TFX pipeline. All artifacts downstream of StatsGen and connected to StatsGen, which includes artifacts unrelated to StatsGen like transform_graph.

https://screenshot.googleplex.com/ey9TJBUYbS1

This issue was reported in the KFP bug bash so follow those instructions for repro.

[BugBash] ExectionType name formatting

child-of #9

About

This issue is regarding the ExecutionName (type header) formatting exceeding the width of the card and overflowing

Solution

  • Clip overflowing text
  • White-space should not wrap
  • On hover show full name

Meta

/area metadata
/area front-end
/priority p0
/assign @avdaredevil
/cc @kwasi

[BugBash] Fix overflow on screens too small to fit lineage view

child-of #9

About

This issue is regarding the lineage view not fitting in most screens (width wise).

Solution

  • Calculate screen width, and dynamically pass in CSS props to calculate edge offsets and card positions
    • An added advantage of auto-centering the target artifact

Meta

/area metadata
/area front-end
/priority p0
/assign @avdaredevil
/cc @kwasi

[MLMD] Pending Issues 2/27 - Kwasi : AP

Missing Features / Bugs

  • Sometimes when a page loads, the MLMD View doesn't load (resizing fixes)
    • Seems to have socket disconnect in console
  • The Artifact Target updates before the new data loaded in (maybe ties into animations)?
  • Many to many edge-linking support is broken in 1st to 2nd column
    • Depends on whether multiple executions can all produce the same artifact

Meta

/area metadata
/area front-end
/priority p2
/assign @avdaredevil @kwasi
/cc @truptinayak

Clean up dependencies

Current dependencies list include a few things not suitable there, like react, react-dom, react-scripts. They should be moved to dev dep and react should be peer too.
/assign @kwasi

[MLMD][Bug Bash] Add links for Artifacts shown on the ExecutionDetails page

/area metadata
/area front-end
/priority p0
/assign @kwasi
/cc @avdaredevil

Background

This is feature request for ArtifactDetails which is a page within the Pipelines application, rather than shared component. Fixing it Pipelines won't fix it in every place where MLMD Lineage is being used so for the purpose of this bug it's enough to fix the is issue in Pipelines in the short term.

This issue is being filed in this repo and not pipelines to track finding a more long-term solution where the Artifact list shown on the ExecutionDetails page can be shared. This could be resolved by upstreaming the whole ArtifactDetails to be shared between metadata and pipelines, but that may require more time given that the page components generally expect routing params from the application.

Tasks

  • (short-term) Update Artifact list on ExecutionDetails page in KFP to use the <ArtifactLink /> component for URIs
  • (short-term) Update Artifact list on ExecutionDetails page in KFP to use link name to Artifact details page
  • (long-term) Upstream ExecutionDetails page artifact list component and use in both kubeflow/pipelines and kubeflow/metadata

[MLMD Lineage] Add RPCs for non-first row execution cards

/kind bug
/area front-end

Issue

When rendering a <LineageCardRow/> for an Execution we always show a next and previous dot.

We should:

  • Make RPCs to request if non-first row Execution cards have any input artifacts
  • Make RPCs to request if non-first row Execution cards have any output artifacts
  • Conditionally render the next and previous dots

Repro Steps

  • This bug happens with all data.

Exclude build artifacts from package via gitignore

Background

As of now I haven't gotten kubeflow/frontend to build successfully in both kubeflow/metadata and kubeflow/pipelines without running into conflicts with the way they both enforce TS config and style rules.

kubeflow/pipelines uses react-scripts-ts
kubeflow/frontend uses reacts-scripts vanilla

These are the ts configs:

https://github.com/kubeflow/metadata/blob/master/frontend/tsconfig.json
https://github.com/kubeflow/pipelines/blob/master/frontend/tsconfig.json

These config files are enforced by react-scripts-ts:

https://github.com/kubeflow/pipelines/blob/master/frontend/tsconfig.prod.json
https://github.com/kubeflow/pipelines/blob/master/frontend/tsconfig.test.json

Problem

This repo includes checked in ES5 code and points instead of compiling as a postinstall step.

Workaound

NOTE: You have to run npm build after making changes in order for the changes to be reflected in dependent repos. npm run build:watch will allow changes to be reflected during development.

Fix Tasks:

  • Update this issue with an output of the build conflict
  • Unify kubeflow/frontend and kubeflow/pipelines TS Config and TS Lint rules so they aren't in conflict.
  • Run npm build as postinstall
  • Add build/ to .gitignore and remove build artifcats

[BugBash] ExectionType name - remove hyperlinking

child-of #9

About

This issue is regarding the ExecutionName formatting looking like a link, but not doing anything, we will snub out this behavior for this release, and consider making it selectable in the next iteration sometime in Q2-3

Solution

  • Remove underline
  • Remove hover styles

Meta

/area metadata
/area front-end
/priority p1
/assign @avdaredevil
/cc @kwasi

[BugBash] Make secondary text on all rows the run-id instead of workspace

child-of #9

About

This issue is regarding the subtext under each metadata row being workspace currently, and we'd like to change it to show run id instead. Reason being: Using Pipeline name as the description makes it hard to distinguish Runs involving the same Artifact.

Solution

  • Workspace_name -> run-id
  • Add workspace name to breadcrumbs bar

Meta

/area metadata
/area front-end
/priority p0
/assign @avdaredevil
/cc @kwasi

Deprecation notice: this repo will be archived and moved to kubeflow/pipelines

Right now, kubeflow/pipelines is the only maintained repo that depends on kubeflow/frontend.
If there are no further objections, I'll archive this repo and move content to github.comkubeflow/frontend about one month later.

I'll keep the directory structure similar and independent, so that in case there's a need to build reusable components again, we can pull them out. However, with current situation, it saves the most efforts if these components live directly in kubeflow/pipelines.

/cc @avdaredevil @jlewi

[MLMD Lineage] Add RPCs for next/previous dots pointing offscreen

/kind bug
/area front-end

Issue

When rendering a <LineageCardRow/>we currently always show:

  • Previous dots for Input Artifacts
  • Next dots for Output Artifacts

We should:

  • Make RPCs to request if the Input Artifacts were created by an execution
  • Make RPCs to request if the Output Artifacts were used in any executions
  • Conditionally render the next and previous dots

Repro Steps

  • This bug happens with all data.

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    ๐Ÿ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. ๐Ÿ“Š๐Ÿ“ˆ๐ŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google โค๏ธ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.