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

Implement solver_infinity for XPRESS and make it available in Java #4313

Closed
wants to merge 1 commit into from

Conversation

pet-mit
Copy link
Contributor

@pet-mit pet-mit commented Jul 15, 2024

@lperron
Copy link
Collaborator

lperron commented Jul 15, 2024

why not use Double.PositiveInfinity ?

@pet-mit
Copy link
Contributor Author

pet-mit commented Jul 15, 2024

why not use Double.PositiveInfinity ?

That's a good point. I guess we thought that using solver_infinity() (since it's in the OR-Tools & solvers' APIs) would be more robust.

@lperron
Copy link
Collaborator

lperron commented Jul 15, 2024 via email

@pet-mit
Copy link
Contributor Author

pet-mit commented Jul 15, 2024

It was an oversight on my part. They both return double +inf. Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53 00 Le lun. 15 juil. 2024 à 17:00, Peter Mitri @.> a écrit :

why not use Double.PositiveInfinity ? That's a good point. I guess we thought that using solver_infinity() (since it's in the OR-Tools & solvers' APIs) would be more robust. — Reply to this email directly, view it on GitHub <#4313 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACUPL3P3CKZ3TTRQMUTE3NDZMPPXRAVCNFSM6AAAAABK4OFCFOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRYG4YTCMZUG4 . You are receiving this because you commented.Message ID: @.
>

so you think we should ignore these methods, and instead use Double.POSITIVE_INFINITY in our Java code?

@pet-mit pet-mit closed this Sep 20, 2024
@Mizux Mizux added this to the v9.12 milestone Nov 5, 2024
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.

3 participants