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

Add a backend option for the computation of deformation cone #39540

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

jplab
Copy link
Contributor

@jplab jplab commented Feb 17, 2025

This is a quick follow-up on #39496 that adds a deformation cone method that encodes regular subdivisions.

With this option, we may speed-up the computations

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation and checked the documentation preview.

⌛ Dependencies

#39496

@fchapoton
Copy link
Contributor

    poly = Polyhedron(ieqs=ieqs, backend)
                                        ^
SyntaxError: positional argument follows keyword argument

@fchapoton
Copy link
Contributor

attention, comme je t'avais dit, j'ai fait des modifs directement sur ta branche sur github sur l'autre ticket. Il aurait fallu tirer ces modifs avant de batir d'autres commits par dessus ta branche. A priori, ca va merger sans probleme, mais on ne sait jamais.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants