Update forked branch-0.18 from release#40
Merged
aschaffer merged 33 commits intoaschaffer:branch-0.18from Jan 12, 2021
Merged
Conversation
This PR includes various updates to support nightly automated MG test runs, including: * Adding a marker which nightly scripts use to run on all visible GPUs instead of a hardcoded number of GPUs, since the scripts rely on knowing the number of GPUs being used in the tests by setting the `CUDA_VISIBLE_DEVICES` env var. * In the nightly scripts, the marker is used like so: `pytest -m "not preset_gpu_count" ...` * Added a `client.wait_for_workers()` call to various setups to both match the approach taken by the `MGContext` class, and to ensure workers are running. This seemed to increase reliability in the test runs. * _side note: we should decide to use only the `MGContext` class or the `client_connection` pytest fixture in these tests, since they both aim to accomplish the same thing._ Authors: - Rick Ratzel <rratzel@nvidia.com> Approvers: - null URL: #1308
…test Fix EXPERIMENTAL_LOUVAIN_TEST on Pascal
MNMG Louvain uses a feature not supported on Pascal. This PR updates the python unit tests so that it expects an exception to be raised if running on a Pascal GPU. Authors: - Charles Hastings <charlesh@nvidia.com> - Chuck Hastings <charlesh@nvidia.com> Approvers: - Brad Rees - Alex Fender - Rick Ratzel URL: #1322
- Add const to input pointers. - Use a double type counter in std::accumulate as std::accumulate is inaccurate in adding a large number of a comparably sized values. - Fix random number generator seed. - Re-enable a temporarily disabled test case. - Relax the thresholds to skip comparison for lowly ranked vertices (with low scores which are more susceptible to the limited floating-point resolution) Authors: - Seunghwa Kang <seunghwak@nvidia.com> Approvers: - Andrei Schaffer (@aschaffer) - Brad Rees (@BradReesWork) URL: #1307
# Summary This PR adds a GitHub action (PR Labeler) to auto label PRs based on a labeling rule and changes in PR. Authors: - Joseph <68436579+jolorunyomi@users.noreply.github.com> Approvers: - AJ Schmidt (@ajschmidt8) - Rick Ratzel (@rlratzel) - AJ Schmidt (@ajschmidt8) URL: #1318
… CI env(#1325) Updated git utils used by copyright.py for compatibility with current CI env, added debug prints. <s>NOTE: I'm intentionally not updating the copyright date on the changed file to test that the check is working in the actual CI environment here. Once verified, I'll change it from a draft PR and mark it ready for review.</s> _verified_ Authors: - Rick Ratzel <rratzel@nvidia.com> Approvers: - Ray Douglass (@raydouglass) URL: #1325
[REVIEW] Fix graph nodes function and renumbering from series
[REVIEW] FIX Set bash trap after PATH is updated [skip-ci]
Fixes #1328 Authors: - Ishika Roy <iroy@nvidia.com> Approvers: - Brad Rees (@BradReesWork) URL: #1326
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update forked branch-0.18 from release