Skip to content

factorial strategy missing in api#490

Open
bertiqwerty wants to merge 1 commit intomainfrom
factorial_api
Open

factorial strategy missing in api#490
bertiqwerty wants to merge 1 commit intomainfrom
factorial_api

Conversation

@bertiqwerty
Copy link
Contributor

No description provided.

@bertiqwerty bertiqwerty requested a review from jduerholt January 2, 2025 09:42
@jduerholt
Copy link
Contributor

jduerholt commented Jan 6, 2025

Failing tests seems to be a py 3.9 issue with entmoot, or? This is also the case in your other PR. Can you have a look into this? From my point of view we could also remove 3.9 support.

Were there any changes in ENTOOT? @TobyBoyne

@TobyBoyne
Copy link
Collaborator

Hi @jduerholt, yes there was a small bug fix in entmoot, and I also pinned he gurobipy version to <12. I would hope that wouldn't introduce more bugs...

From looking at the tests, I can see that many are failing due to using | to combine types instead of Union, which isn't supported in py3.9 - is this what is causing issues?

@jduerholt
Copy link
Contributor

jduerholt commented Jan 6, 2025

Yes, exactly this is causing the errors. @bertiqwerty: should we drop the support for 3.9?

@jduerholt
Copy link
Contributor

FactorialStrategy is deprecated, FractionalFactorialStrategy can do exactly the same.

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