- 
                Notifications
    
You must be signed in to change notification settings  - Fork 7
 
Pairwise #255
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
base: main
Are you sure you want to change the base?
Pairwise #255
Conversation
        
          
                src/frank_wolfe_variants.jl
              
                Outdated
          
        
      | 
               | 
          ||
| Base.print(io::IO, ::StandardFrankWolfe) = print(io, "StandardFrank-Wolfe") | ||
| 
               | 
          ||
| @info "After define" PairwiseFrankWolfe | 
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.
Please remove
        
          
                src/frank_wolfe_variants.jl
              
                Outdated
          
        
      | return x, primal, dual_gap, active_set | ||
| end | ||
| 
               | 
          ||
| Base.print(io::IO, ::PairwiseFrankWolfe) = print(io, "Pairwise FrankWolfe") | 
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.
Frank-Wolfe
        
          
                src/frank_wolfe_variants.jl
              
                Outdated
          
        
      | Pairwise FrankWolfe | ||
| """ | ||
| 
               | 
          ||
| struct PairwiseFrankWolfe <: FrankWolfeVariant end | 
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.
Please move it between BCG and BPCG
          Codecov Report❌ Patch coverage is  
 Additional details and impacted files@@            Coverage Diff             @@
##             main     #255      +/-   ##
==========================================
- Coverage   86.23%   82.46%   -3.78%     
==========================================
  Files          17       22       +5     
  Lines        1482     2817    +1335     
==========================================
+ Hits         1278     2323    +1045     
- Misses        204      494     +290     ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
  | 
    
Add the Pairwise Conditional Gradient (in pairwise.jl in FrankWolfe.jl) to Boscia