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.

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

Add test cases for communication alone and mixed compute-communication case #533

Closed
ppebay opened this issue Sep 17, 2024 · 0 comments · Fixed by #541
Closed

Add test cases for communication alone and mixed compute-communication case #533

ppebay opened this issue Sep 17, 2024 · 0 comments · Fixed by #541
Assignees
Labels
enhancement New feature or request

Comments

@ppebay
Copy link
Contributor

ppebay commented Sep 17, 2024

  • Test case with beta=1.0, everything else at 0. with recursive strategy:
    We should find W_max=0.0

  • Test case with alpha=beta=1.0, with recursive strategy:

366569915-fb08eff9-0101-4790-98dc-74be04d864b6_720

Imb = L_max / L_ave - 1.0 = 2.5 / 2.0 - 1.0 = 0.25
W_max = 4.0

@ppebay ppebay added the enhancement New feature or request label Sep 17, 2024
@ppebay ppebay changed the title Add test case for mixed compute-communication case Add test cases for communication alone and mixed compute-communication case Sep 17, 2024
cwschilly added a commit that referenced this issue Sep 23, 2024
cwschilly added a commit that referenced this issue Sep 26, 2024
cwschilly added a commit that referenced this issue Sep 30, 2024
…ication case (#541)

* #533: initial commit; add new tests for synthetic blocks dataset

* #533: update with more changes

* #533: update tests to use W_max instead of of load imbalance

* #533: set BFO to false; increase n_iterations

* #533: update logic to avoid writing configuration files

* #533: loosen constraints on W_max in tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants