Skip to content
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

Zoltan2: small coloring cleanup #9268

Merged
merged 1 commit into from
Aug 23, 2021
Merged

Zoltan2: small coloring cleanup #9268

merged 1 commit into from
Aug 23, 2021

Conversation

kddevin
Copy link
Contributor

@kddevin kddevin commented Jun 10, 2021

@trilinos/zoltan2

Motivation

This PR enables tests that were needlessly disabled when Galeri is not built; the tests run without Galeri.
This PR also makes colorMethod names in coloring1.cpp test consistent with the actual coloring method names.

Stakeholder Feedback

Testing

tested on cee-lan with gcc

@kddevin
zoltan2: made method names in test consistent with actual method names;
eed8980 
enabled tests that don't rely on Galeri when Galeri is not built
@kddevin kddevin added pkg: Zoltan2 AT: AUTOMERGE Causes the PR autotester to automatically merge the PR branch once approvals are completed labels Jun 10, 2021
@kddevin kddevin requested a review from egboman June 10, 2021 20:31
@kddevin kddevin self-assigned this Jun 10, 2021
@trilinos-autotester
Copy link
Contributor

Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection Is Not Necessary for this Pull Request.

@trilinos-autotester
Copy link
Contributor

Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects:

Pull Request Auto Testing STARTING (click to expand)

Build Information

Test Name: Trilinos_pullrequest_gcc_8.3.0

  • Build Num: 4667
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_gcc_7.2.0_serial

  • Build Num: 2197
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_gcc_7.2.0_debug

  • Build Num: 2678
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_intel_17.0.1

  • Build Num: 9986
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_cuda_10.1.105

  • Build Num: 1398
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_cuda_10.1.105_uvm_off

  • Build Num: 395
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_clang_10.0.0

  • Build Num: 2763
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_python_3

  • Build Num: 5332
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Using Repos:

Repo: TRILINOS (trilinos/Trilinos)
  • Branch: zoltan2_smallColoringCleanup
  • SHA: eed8980
  • Mode: TEST_REPO

Pull Request Author: kddevin

@trilinos-autotester
Copy link
Contributor

Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED

Pull Request Auto Testing has PASSED (click to expand)

Build Information

Test Name: Trilinos_pullrequest_gcc_8.3.0

  • Build Num: 4667
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_gcc_7.2.0_serial

  • Build Num: 2197
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_gcc_7.2.0_debug

  • Build Num: 2678
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_intel_17.0.1

  • Build Num: 9986
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_cuda_10.1.105

  • Build Num: 1398
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_cuda_10.1.105_uvm_off

  • Build Num: 395
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_clang_10.0.0

  • Build Num: 2763
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5

Build Information

Test Name: Trilinos_pullrequest_python_3

  • Build Num: 5332
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
PR_LABELS AT: AUTOMERGE;pkg: Zoltan2
PULLREQUESTNUM 9268
TEST_REPO_ALIAS TRILINOS
TRILINOS_SOURCE_BRANCH zoltan2_smallColoringCleanup
TRILINOS_SOURCE_REPO https://github.com/trilinos/Trilinos
TRILINOS_SOURCE_SHA eed8980
TRILINOS_TARGET_BRANCH develop
TRILINOS_TARGET_REPO https://github.com/trilinos/Trilinos
TRILINOS_TARGET_SHA b0ec8e5


CDash Test Results for PR# 9268.

@trilinos-autotester
Copy link
Contributor

Status Flag 'Pre-Merge Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
NO REVIEWS HAVE BEEN PERFORMED ON THIS PULL REQUEST!

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur...

1 similar comment
@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur...

@trilinos-autotester trilinos-autotester added the AT: STALE Added by the PR autotester if too much time has elapsed since the last successful PR test iteration label Jun 16, 2021
@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

4 similar comments
@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

egboman
egboman approved these changes Jun 21, 2021
View reviewed changes
Copy link
Contributor

@egboman egboman left a comment

Choose a reason for hiding this comment

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

Good change. Thanks, Karen.

@trilinos-autotester
Copy link
Contributor

Status Flag 'Pre-Merge Inspection' - SUCCESS: The last commit to this Pull Request has been INSPECTED AND APPROVED by [ egboman ]!

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

10 similar comments
@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

22 similar comments
@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@trilinos-autotester
Copy link
Contributor

All Jobs Finished; status = PASSED, However PR is now STALE, and must be retested. Set the AT: RETEST Label to force retest....

@kddevin kddevin added AT: RETEST Causes the PR autotester to run a new round of PR tests on the next iteration and removed AT: STALE Added by the PR autotester if too much time has elapsed since the last successful PR test iteration labels Aug 23, 2021
@kddevin kddevin merged commit 353c286 into develop Aug 23, 2021
jmgate pushed a commit to tcad-charon/Trilinos that referenced this pull request Aug 24, 2021
Merge 'trilinos/Trilinos:develop' (353c286) into 'tcad-charon/Trilino…
e097917 
…s:develop' (5498f28).

* trilinos-develop:
  zoltan2:  made method names in test consistent with actual method names; (trilinos#9268)
  Fix Panzer's tests with UVM off (trilinos#9599)
  MueLu: add doc, remove commented code, formatting
jmgate pushed a commit to tcad-charon/Trilinos that referenced this pull request Aug 24, 2021
Merge 'trilinos/Trilinos:develop' (353c286) into 'tcad-charon/Trilino…
5ef4b4e 
…s:develop' (5498f28).

* trilinos-develop:
  zoltan2:  made method names in test consistent with actual method names; (trilinos#9268)
  Fix Panzer's tests with UVM off (trilinos#9599)
  MueLu: add doc, remove commented code, formatting
@kddevin kddevin deleted the zoltan2_smallColoringCleanup branch November 1, 2021 16:18
PhilMiller pushed a commit to PhilMiller/Trilinos that referenced this pull request Dec 22, 2021
Merge 'trilinos/Trilinos:develop' (353c286) into 'EM-Plasma/Trilinos:…
99d46b6 
…develop' (aaebcf8).

* trilinos/develop:
  zoltan2:  made method names in test consistent with actual method names; (trilinos#9268)
  MueLu: add doc, remove commented code, formatting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@egboman egboman egboman approved these changes

Assignees

@kddevin kddevin

Labels
AT: AUTOMERGE Causes the PR autotester to automatically merge the PR branch once approvals are completed AT: RETEST Causes the PR autotester to run a new round of PR tests on the next iteration pkg: Zoltan2
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kddevin @trilinos-autotester @egboman