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

Python CP-SAT type hints #3668

Merged
merged 1 commit into from Jul 9, 2023
Merged

Python CP-SAT type hints #3668

merged 1 commit into from Jul 9, 2023

Conversation

ghost
Copy link

@ghost ghost commented Feb 7, 2023

Work in progress, opened as draft pull request.

Tasks:

  • Custom types: Integral (same logic as cmh.is_integral), Literal (IntVar+_NotBooleanVariable+Integral), Arc, etc
  • Run mypy on ortools/sat/samples

@oulianov
Copy link

oulianov commented Feb 8, 2023

Thank you for opening this review ! That's a great start.

I added some comments right now with additional typing from the top of my mind. But there are many more missing, even in other files 🤔

What would be the best way to help you cover all the definitions ?

@ghost
Copy link
Author

ghost commented Feb 8, 2023

Thanks for the help offer and suggestions! @oulianov

But I believe I just need some time to figure out the best type definitions for some of the more complex interfaces, eg:

  • OnlyEnforceIf
  • arcs (iterable of sequence of len 3)
  • etc

And test them thoroughly.

@oulianov
Copy link

oulianov commented Feb 8, 2023

For sure, totally understand. Wish you the best, feel free to reach out

@ghost ghost marked this pull request as ready for review February 13, 2023 22:43
@Mizux Mizux added this to the v9.6 milestone Feb 14, 2023
@Mizux Mizux added Feature Request Missing Feature/Wrapper Lang: Python Python wrapper issue Solver: CP-SAT Solver Relates to the CP-SAT solver labels Feb 14, 2023
@ghost ghost marked this pull request as draft February 25, 2023 00:28
@ghost ghost marked this pull request as ready for review March 7, 2023 22:01
@Mizux Mizux modified the milestones: v9.6, v9.7 Mar 13, 2023
@lperron
Copy link
Collaborator

lperron commented Jul 9, 2023

Hi,

Any change of pushing this ?
Just run the black formatter on the code before committing.

@ghost
Copy link
Author

ghost commented Jul 9, 2023

My bad, never saw the conflicts, should be ready to review/merge now

@lperron lperron merged commit 2a07d25 into google:main Jul 9, 2023
108 of 131 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request Missing Feature/Wrapper Lang: Python Python wrapper issue Solver: CP-SAT Solver Relates to the CP-SAT solver
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants