question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Investigate SAG/SAGA solver

See original GitHub issue

Description

The newly introduced tight tests for Ridge in #22910 together with the random seed fixture in #22749 revealed some shortcomings of the sag and saga solver:

  1. It shows some random behavior even with fixed random seed.
  2. The tol needs to be set much smaller to receive comparable results with the other solvers of Ridge.

Ideally, the cause for both issues can be identified and fixed.

Some links for context

https://github.com/scikit-learn/scikit-learn/issues/23014 https://github.com/scikit-learn/scikit-learn/pull/23017 https://github.com/scikit-learn/scikit-learn/pull/23026 https://github.com/scikit-learn/scikit-learn/pull/23152 #23177

Issue Analytics

  • State:open
  • Created a year ago
  • Comments:6 (6 by maintainers)

github_iconTop GitHub Comments

1reaction
TomDLTcommented, Dec 2, 2022

could this be connected to #24679 ?

No. The linked issue is a potential mistake on the prox operator in SAGA. However, the prox operator is not used in Ridge, because the prox operator is only used for L1 regularization. Therefore, this potential mistake should not affect Ridge convergence.

edit: #24679 was not a mistake.

0reactions
glevvcommented, Oct 28, 2022

could this be connected to #24679 ?

Read more comments on GitHub >

github_iconTop Results From Across the Web

Mrs Johnson's Storyline - Summertime Saga Wiki Guide - IGN
This page takes you through Mrs Johnson's storyline which will see you get to know Mrs Johnson while helping Erik out at school....
Read more >
LogisticRegression with SAGA using sample_weight does not ...
We have a common test that checks this here but we are currently not running it for non default solvers, I think. We...
Read more >
ABC57 Investigates
ABC57 News in South Bend, Ind. covers all of Michiana including St. Joseph, Elkhart, Kosciusko, LaPorte and Marshall counties in Indiana and Berrien,...
Read more >
Comparison of the ease of use and accuracy of two machine ...
The different solvers for it were compared with and without normalization. (Table 1). Normalization improved the performance of the sag and saga solvers,...
Read more >
Genshin Impact The Saga Of Mr. Forgetful World Quest Guide
Forgetful, rewards, torch puzzle, take a photo of the ruin murals! Table of Contents. How To Unlock? Walkthrough Chart & Tips; Rewards. 2.2 ......
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found