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

#13: Debug incorrect reported values of W when beta is non zero #14

Merged

Conversation

ppebay
Copy link
Contributor

@ppebay ppebay commented Sep 16, 2024

Fixes #13

  • also adds sanity checks (between $\psi$ and $\chi$
  • improves printouts for clarity of results

Specifically for python ccm_milp_full.py -c ../tests/config/1-fwmp-with-alpha-beta.yaml, the obtained configuration:

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

now gives the correct values for W:

# Solution summary:
Rank 0: L = 2.5, W = 4.0, unhomed: 1
Rank 1: L = 1.5, W = 4.0, unhomed: 1
Rank 2: L = 2.0, W = 4.0, unhomed: 1
Rank 3: L = 2.0, W = 4.0, unhomed: 2

Copy link
Contributor

@maxime-bfsquall maxime-bfsquall left a comment

Choose a reason for hiding this comment

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

I just make a commit for pylint issue but nothing else to report

@ppebay ppebay merged commit da51371 into master Sep 17, 2024
9 checks passed
@ppebay ppebay deleted the 13-debug-incorrect-reported-values-of-W-when-beta-is-non-zero branch September 17, 2024 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Debug incorrect reported values of W when beta is non zero
2 participants