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

More SpineOpt #38

Merged
merged 4 commits into from
Nov 26, 2024
Merged

More SpineOpt #38

merged 4 commits into from
Nov 26, 2024

Conversation

joaquimg
Copy link
Member

@joaquimg
Copy link
Member Author

There is something non-deterministic.
Every time I run I get a different Hash.
Is this expected @datejada ?

@joaquimg
Copy link
Member Author

Row labels get messed up:
image

@datejada
Copy link
Contributor

There is something non-deterministic. Every time I run I get a different Hash. Is this expected @datejada ?

@joaquimg I'm not aware that something is non-deterministic. @manuelma or @Tasqu, do you have any idea about this behaviour in SpineOpt? Basically, the order of the constraints in the .mps file seems to change for each run.

@joaquimg
Copy link
Member Author

@odow, can you test running the SpineOpt script on your side to see if everything is properly documented?

@Tasqu
Copy link

Tasqu commented Nov 21, 2024

@datejada I'm not surprised it's non-deterministic. We use Julia Dicts in SpineOpt quite a bit and their ordering is not consistent. Furthermore, I talked with Antti, and depending on how SpineInterface queries the databases it might also go through Python dictionaries which are not ordered either.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@joaquimg
Copy link
Member Author

I am merging this, as it is a strict improvement over what we currently have.

@joaquimg joaquimg merged commit e647469 into main Nov 26, 2024
5 checks passed
@joaquimg joaquimg deleted the jg/spine branch November 26, 2024 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants