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

[docs] add p-norm example #3282

Merged
merged 4 commits into from
Mar 13, 2023
Merged

[docs] add p-norm example #3282

merged 4 commits into from
Mar 13, 2023

Conversation

@codecov
Copy link

codecov bot commented Mar 13, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.13 ⚠️

Comparison is base (f38bfce) 98.11% compared to head (809e166) 97.98%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3282      +/-   ##
==========================================
- Coverage   98.11%   97.98%   -0.13%     
==========================================
  Files          34       34              
  Lines        4714     4714              
==========================================
- Hits         4625     4619       -6     
- Misses         89       95       +6     

see 6 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

@odow
Copy link
Member Author

odow commented Mar 13, 2023

This probably comes up often enough that I wonder if it's worth a dedicated set and bridge?

@blegat
Copy link
Member

blegat commented Mar 13, 2023

This probably comes up often enough that I wonder if it's worth a dedicated set and bridge?

Yes, @chriscoey do you have something like that in Hypatia ?

@odow
Copy link
Member Author

odow commented Mar 13, 2023

I've opened an issue: jump-dev/MathOptInterface.jl#2118.

This doc PR can be merged without waiting for the resolution.

@chriscoey
Copy link
Contributor

This probably comes up often enough that I wonder if it's worth a dedicated set and bridge?

Yes, @chriscoey do you have something like that in Hypatia ?

No, we didn't know a nice barrier for p-norm so we didn't try implementing anything.

@odow odow merged commit e474981 into master Mar 13, 2023
@odow odow deleted the od/p-norm branch March 13, 2023 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants