Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

[WebUI] support tensorboard #3361

Merged
merged 19 commits into from
Apr 9, 2021
Merged

[WebUI] support tensorboard #3361

merged 19 commits into from
Apr 9, 2021

Conversation

Lijiaoa
Copy link
Contributor

@Lijiaoa Lijiaoa commented Feb 4, 2021 •
edited
Loading

related design issue: #3323

This PR still has some questions to fix:

  • about clear timeout function: 1. stop all tensorboard; 2. /detail -> /oview change page
  • add tensorboard start failed dialog
  • refactor【repeated tensorboard dialog and tensorboard detail dialog use same component】
  • use childrenGap instead of gap to fix console warning
  • test

@Lijiaoa Lijiaoa marked this pull request as draft February 4, 2021 06:48
@Lijiaoa Lijiaoa changed the title [For next release] support tensorboard [TODO] support tensorboard Mar 17, 2021
@Lijiaoa Lijiaoa changed the title [TODO] support tensorboard [WebUI] support tensorboard Apr 7, 2021
@Lijiaoa Lijiaoa marked this pull request as ready for review April 7, 2021 02:07
@SparkSnail SparkSnail mentioned this pull request Apr 7, 2021
Closed
78 tasks
@SparkSnail SparkSnail merged commit aec4ce1 into microsoft:master Apr 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@J-shang J-shang J-shang approved these changes

@ultmaster ultmaster ultmaster approved these changes

@liuzhe-lz liuzhe-lz Awaiting requested review from liuzhe-lz

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Lijiaoa @ultmaster @J-shang @SparkSnail