Skip to content

[test] Update AllocSpeed check for Kesch #1098

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Dec 10, 2019
Merged

[test] Update AllocSpeed check for Kesch #1098

merged 2 commits into from
Dec 10, 2019

Conversation

@victorusu
Copy link
Contributor

@victorusu victorusu commented Dec 9, 2019 •
edited by vkarak
Loading

Fixes UES-660.

@victorusu victorusu requested review from kraushm and vkarak December 9, 2019 16:09
@victorusu victorusu self-assigned this Dec 9, 2019
kraushm
kraushm approved these changes Dec 9, 2019
View reviewed changes
Copy link
Contributor

@kraushm kraushm left a comment

Choose a reason for hiding this comment

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

lgtm

vkarak
vkarak requested changes Dec 9, 2019
View reviewed changes
Copy link
Contributor

@vkarak vkarak left a comment •
edited
Loading

Choose a reason for hiding this comment

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

I think you should update the reference for kesch:pn, too.

@vkarak vkarak added this to the ReFrame sprint 2019w48 milestone Dec 10, 2019
@victorusu victorusu force-pushed the test/alloc_speed_test branch from 685f4ec to 24033bb Compare December 10, 2019 09:44
@vkarak vkarak changed the title [test] Fix kesch performance reference AllocSpeed_no [test] Update AllocSpeed check for Kesch Dec 10, 2019
@vkarak vkarak merged commit c8de1dd into reframe-hpc:master Dec 10, 2019
@victorusu victorusu deleted the test/alloc_speed_test branch October 6, 2020 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vkarak vkarak vkarak approved these changes

+1 more reviewer

@kraushm kraushm kraushm approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@victorusu victorusu

Labels

prio: important

Projects

None yet

Milestone

ReFrame sprint 2019w48

Development

Successfully merging this pull request may close these issues.

3 participants

@victorusu @vkarak @kraushm