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

Make selected trials consistent across auto-refresh in detail table #3597

Merged
merged 5 commits into from
May 17, 2021

Conversation

Lijiaoa
Copy link
Contributor

@Lijiaoa Lijiaoa commented Apr 30, 2021

PR content:
修的问题:

  • 选中的 trial list 会随着页面的刷新被置空
  • 在某一页选中 trial ,切换到另一页,之前选中的 trial 被清空
  • related issue

related changes:

  • 选中 trial 之后,做了 tensorboard 操作之后,清空选中的 trial 列表
  • 选中 trial 之后,做了 compare 操作之后,清空选中的 trial 列表(compare modal file 支持 单 trial intermediate graph 和 trial compare 两种场景,仅在 trial compare 下做清空操作)

测试用例:

启动一个基本实验(trial 条数 > 10 条),查看 /detail 页面

(1)在 table 中选择几条 trial,观察页面定时刷新时(可以观察 Duration 列来判断是否刷新了)这几条 trial 的选中状态,如果依然是被选中,测试正确,否则,有问题

(2)在(1)的基础上,单机 compare 按钮,会弹出 compare modal,此时,点击页面空白处来关闭 compare modal,table 里选中的 trail 自动被清掉了,测试正确

(3)在(1)的基础上,单机 compare 按钮,会弹出 compare modal,此时,点击 compare modal 框里的 × 按钮来关闭 compare modal,table 里选中的 trial 自动被清掉了,测试正确

(4)在(1)的基础上,单机 tensorboard 按钮,关闭 tensorboard modal 之后,table 里选中的 trial 自动被清掉了,测试正确

(5)使用 table 分页功能,在不同的页里进行 select trial 操作,再翻看之前页里选中的 trial 是不是依然是选中的状态,是选中状态,测试正确

@ultmaster ultmaster self-requested a review May 7, 2021 08:41
@ultmaster ultmaster requested a review from liuzhe-lz May 7, 2021 08:42
@Lijiaoa Lijiaoa closed this May 10, 2021
@Lijiaoa Lijiaoa reopened this May 10, 2021
}
items.forEach(item => {
if (item.id === id) {
item.checked = !!checked;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why double !

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

follow the example on fluent doc, look like this picture
image

@ultmaster ultmaster merged commit e5fb9c5 into microsoft:master May 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make selected trials consistent across auto-refresh in detail table
3 participants