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

Inconsistencies when using random_instance() to generate a random instance of a problem class #103

Open
Q-lds opened this issue Oct 18, 2022 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@Q-lds
Copy link
Member

Q-lds commented Oct 18, 2022

Problem

When trying to create a random instance of a number partitioning problem, I get the following error

image

The core issue is that random_instance() is a static method and should be used as such:
image

The inconsistency is made worst by the fact that I can do the following, where the numbers I pass to the constructor are right away changed by the random_instance() method

image

#Solution

I am unsure what's the best way to go about this, but we should review the way that random instances are created across the pre-defined problem settings.

@Q-lds Q-lds added the bug Something isn't working label Oct 18, 2022
@Q-lds
Copy link
Member Author

Q-lds commented Dec 16, 2022

@vishal-ph and @Adirlou I think we can close this, right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants