Skip to content

Conversation

@geoffroyleconte
Copy link
Member

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 1, 2023

Here are the
demos-results

@geoffroyleconte geoffroyleconte requested a review from dpo June 1, 2023 16:48
@codecov
Copy link

codecov bot commented Jun 1, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (ea5a650) 59.34% compared to head (08439f1) 59.34%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #104   +/-   ##
=======================================
  Coverage   59.34%   59.34%           
=======================================
  Files          11       11           
  Lines        1225     1225           
=======================================
  Hits          727      727           
  Misses        498      498           

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

reset!(f)
TR_out = TR(f, h, χ, options, selected = selected)
plot_nnmf(TR_out, Avec, m, n, k, "tr-r2-$suffix")

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the reason for removing these lines?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They are already written a few line above. I must have done a wrong copy/paste when adding this demo.

@geoffroyleconte geoffroyleconte merged commit 834bcbc into JuliaSmoothOptimizers:master Jun 5, 2023
@geoffroyleconte geoffroyleconte deleted the errors-demos branch June 5, 2023 14:22
@github-actions
Copy link
Contributor

github-actions bot commented Jun 5, 2023

Here are the
demos-results

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants