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

account for (xexp, mosek) failure in test_constant_atoms.py #1983

Closed
rileyjmurray opened this issue Dec 24, 2022 · 0 comments
Closed

account for (xexp, mosek) failure in test_constant_atoms.py #1983

rileyjmurray opened this issue Dec 24, 2022 · 0 comments

Comments

@rileyjmurray
Copy link
Collaborator

In a recent PR of mine, I noticed that testing the composite atom xexp(pos(x)) failed with MOSEK in test_constant_atoms.py. I tried to add this (atom, solver) pair to a list of known exclusions, via the KNOWN_SOLVER_ERRORS list already in test_constant_atoms.py. I now see that the way I did this was insufficient, since it didn't work with composite atoms. I'm preparing a PR that will change test_constant_atoms.py to be more robust to this kind of thing.

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

No branches or pull requests

1 participant