Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ Transurgeon
✅ SteveDiamond
❌ William Zijie Zhang

What's the plan for projecting onto multiple attributes?

What's the plan for projecting onto multiple attributes?

To start, projection will be disabled when there are multiple attributes. In the long run there may be a few combinations of attributes that are still worth projecting onto.

ok I had another look at this and was a bit more thoughtful with tests, cases like sparse integer, sparse bounded, and integer bounded variables all seem to work.

One thing that concerns me is people might want to pass sparse values as bounds (or only the nonzeros), so we will need to add a bit of code to cover that. But maybe we can do that in another PR if this seems fine for now?

ok I had another look at this and was a bit more thoughtful with tests, cases like sparse integer, sparse bounded, and integer bounded variables all seem to work.

One thing that concerns me is people might want to pass sparse values as bounds (or only the nonzeros), so we will need to add a bit of code to cover that. But maybe we can do that in another PR if this seems fine for now?

Yeah, we can worry about sparse bounds later.

I added tests for parameters with multiple attributes. These tests exposed an issue where parameter input validation doesn't work for multiple attributes. The issue is too complex to fix in this MR.

I added tests for parameters with multiple attributes. These tests exposed an issue where parameter input validation doesn't work for multiple attributes. The issue is too complex to fix in this MR.

ok sounds good, should we add some expected to fail decorators to those tests? And add TODOs for later?

@Transurgeon I fixed the tests but one of the benchmarks is failing

made a PR to fix the failing (and unrelated) test #2784 .
also based on a new comment in discussion #2664, I think we should also test if sparse complex variables work. However, I can't really think of a good test. copilot suggested something but I am not sure its making any sense.

made a PR to fix the failing (and unrelated) test #2784 . also based on a new comment in discussion #2664, I think we should also test if sparse complex variables work. However, I can't really think of a good test. copilot suggested something but I am not sure its making any sense.

I added complex + sparse tests. They actually caught a bug.