Closed marvinvanaalst closed 1 year ago
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.
@marvinvanaalst, and sorry for the delay in getting to this, and thanks a bunch for the PR.
2 things before we merge this in:
pyproject.toml
regarding the black
configuration? (since its outside the scope of this PR).np.int
pointing to int
has been in numpy since version 1.7, the current min. version of numpy
for osqp
. So while I agree that np.int
should be replaced
with int, there's no need to bump up the required numpy version. If you can change that back in requirements.txt
that would be the right thing to do.@marvinvanaalst - after approving PR #109 with this fix, I'm closing this PR. Release v0.6.2.post9 of osqp incorporates these changes. Thanks for bringing this to our attention!
This changes the
np.int
alias of the Python builtinint
type as the alias was deprecated in numpy version 1.20.0 and lead to anAttributeError
starting with version 1.24.0.This should fix issue #104