Skip to content
This repository has been archived by the owner on Jul 17, 2024. It is now read-only.

feat: Add support for enterprise edition and multithreaded solving #31

Merged
merged 4 commits into from
Apr 15, 2024
Merged

feat: Add support for enterprise edition and multithreaded solving #31

merged 4 commits into from
Apr 15, 2024

Conversation

Christopher-Chianelli
Copy link
Collaborator

@Christopher-Chianelli Christopher-Chianelli commented Apr 10, 2024

  • Need to add @PlanningId to Python immutable primitive types in case they are used in value ranges

  • Replace timefold-solver with timefold-solver-enterprise (which has timefold-solver as a dependency).

  • Add tests for the exception raised when an enterprise feature is requested and enterprise is not installed

  • Add support for multithreaded solving in the SolverConfig

- Need to add @PlanningId to Python immutable primitive types in case they are
  used in value ranges

- Two ways of installing enterprise edition:

  - Specify the `enterprise` feature: `timefold-solver[enterprise]`,
    which adds `timefold-solver-enterprise` as a dependency.

  - Replace `timefold-solver` with `timefold-solver-enterprise` (which
    has `timefold-solver` as a dependency).

- Add tests for the exception raised when an enterprise feature is
  requested and enterprise is not installed

- Add support for multithreaded solving in the SolverConfig
Copy link
Contributor

@triceo triceo left a comment

Choose a reason for hiding this comment

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

The ID situation needs to be solved better.

Also, would it simplify anything if we had only one way of enabling enterprise, instead of two? I wouldn't mind if the only supported way was to switch community for enterprise; no extra dependencies.

@Christopher-Chianelli Christopher-Chianelli merged commit 158aca7 into TimefoldAI:main Apr 15, 2024
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants