Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ENH migrate GLMs / TweedieRegressor to linear loss #22548
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
ENH migrate GLMs / TweedieRegressor to linear loss #22548
Changes from all commits
decbb5a
26c0836
fb66241
395743a
c029dad
2eb28f4
fd8d1ca
09b243a
cecb328
719bb1b
5f56968
7adc1bc
2c13683
02dd2de
5a455b0
dda943f
9559464
85f616f
bc19cf3
9582e8b
3b6beda
0ebe6f2
e314ad9
6b25501
8c6e08d
324b858
d5f4723
80bf470
5c9b264
db6c192
a0be233
225863b
38b3104
4bfc1e9
7f428b5
a1474d0
2ca9586
f23dd31
129bce5
db4d402
87dd217
238bf6e
5170aca
111dd8a
5a5a22f
16e8be5
0024ca3
49fb93f
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
My curiosity: when does it make sense to use identity link with power != 0 ?
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.
Years ago, I thought it a good idea. Meanwhile, I don't think it's is useful. Therefore, I opened #19086 without much response.
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.
I have a problem where I know the expectation y' follows the linear model y' = w x. My measurements, y, have poisson errors.
(The specific problem involves analysis of radiation measurements. The expectation is linear with the amount of source; the measurements are poisson distributed).
Using a log link function is just not the right description of my problem. Yes, the whole thing breaks down when evaluating negative values of w, but it seems much better to offer a constraint to avoid ever evaluating negative values of w rather than exclude the situations where you have an actual linear relationship with poisson measurements.
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.
This new loss class is needed for
TweedieRegressor(link="identity")
.See Cython implementation in file
_loss.pyx.tp
.