-
Notifications
You must be signed in to change notification settings - Fork 20
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 Artifacts.toml #338
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #338 +/- ##
===========================================
- Coverage 89.11% 62.29% -26.83%
===========================================
Files 5 7 +2
Lines 790 1151 +361
===========================================
+ Hits 704 717 +13
- Misses 86 434 +348 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@tmigot I added an optional argument to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still good for me
03f5228
into
JuliaSmoothOptimizers:main
Related to #337
sifcollection
added yesterday by Nick.netlib-lp
andmaros-meszaros
).lazy=true
to only download an artifact if we need it.For example, we should not download any artifact if the user defines the environment variable
MASTSIF
.Note that the new standalone decoder available in
SIFDecode_jll.jl
and used sinceCUTEst_jll.jl
v0.13.3 is tested on all these problems: ralna/SIFDecode#20I tested it with
single
,double
andquadruple
precision.We could update the function
set_matsif
to provide an option to switch betweensifcollection
,maros-meszaros
andnetlib-lp
sets in a future PR.