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

Update installation.md #3387

Merged
merged 2 commits into from May 26, 2023
Merged

Update installation.md #3387

merged 2 commits into from May 26, 2023

Conversation

bodono
Copy link
Contributor

@bodono bodono commented May 25, 2023

Since version 3.0 SCS can handle QPs natively (ie, without transforming to SOCP).

Since version 3.0 SCS can handle QPs natively (ie, without transforming to SOCP).
docs/src/installation.md Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented May 25, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (4a304dd) 98.06% compared to head (8fd9263) 98.06%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3387   +/-   ##
=======================================
  Coverage   98.06%   98.06%           
=======================================
  Files          34       34           
  Lines        4922     4922           
=======================================
  Hits         4827     4827           
  Misses         95       95           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Co-authored-by: Benoît Legat <benoit.legat@gmail.com>
@odow
Copy link
Member

odow commented May 25, 2023

I don't think we've updated SCS.jl to support this. So while it's true for SCS, it isn't true for SCS.jl. But it'll still work via bridges, so maybe this is okay anyway.

@odow
Copy link
Member

odow commented May 25, 2023

Opened an issue to track: jump-dev/SCS.jl#273. Okay merging this for now.

@odow odow merged commit a1d2ce6 into jump-dev:master May 26, 2023
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants