-
Notifications
You must be signed in to change notification settings - Fork 84
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
MRG: Pyglmnet paper #313
MRG: Pyglmnet paper #313
Conversation
Codecov Report
@@ Coverage Diff @@
## master #313 +/- ##
=======================================
Coverage 75.66% 75.66%
=======================================
Files 4 4
Lines 678 678
Branches 149 149
=======================================
Hits 513 513
Misses 128 128
Partials 37 37 Continue to review full report at Codecov.
|
Thanks @jasmainak ! Looks good to me |
f27170f
to
f8dc2e2
Compare
hi @themantalope can you read the first comment and confirm your authorship by checking the affiliation and adding the orcid? Thanks. |
I will be completely out until next thursday. Sorry guys. Huge grant deadlin
…On Fri, Nov 8, 2019 at 9:28 AM Mainak Jas ***@***.***> wrote:
hi @themantalope <https://github.com/themantalope> can you read the first
comment <#313 (comment)>
and confirm your authorship by checking the affiliation and adding the
orcid? Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#313?email_source=notifications&email_token=AA2XUNONXBZZ7SYJZZ5U263QSVZQ5A5CNFSM4IYD6K42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDSIUGI#issuecomment-551848473>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA2XUNLV7FJQAEYXIQTOU63QSVZQ5ANCNFSM4IYD6K4Q>
.
--
I love those that desire the impossible - Goethe.
Konrad Kording @KordingLab, UPenn, Richards 404b
cell:773 - liberat (5423728)
office:(215) 898-5826
kordinglab.com
|
no worries, we are expecting you to make at least two pull requests since you are taking extra time now ;) |
I have done my orcid ;) so just one?
…On Fri, Nov 8, 2019 at 4:08 PM Mainak Jas ***@***.***> wrote:
no worries, we are expecting you to make at least two pull requests since
you are taking extra time now ;)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#313?email_source=notifications&email_token=AA2XUNIK66ITPVV664WPFTLQSXILHA5CNFSM4IYD6K42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDTLG2Q#issuecomment-551990122>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA2XUNKATT3WYWNDI7G5G43QSXILHANCNFSM4IYD6K4Q>
.
--
I love those that desire the impossible - Goethe.
Konrad Kording @KordingLab, UPenn, Richards 404b
cell:773 - liberat (5423728)
office:(215) 898-5826
kordinglab.com
|
@AnchorBlues @BeibinLi @DreHar @the872 @themantalope @marquesVF @evadyer @tommyod @ravigarg27 @timshell I have not heard from you in this thread. We do need you to confirm your name and affiliation if you want to be included as an author. |
@jasmainak I am in a conference with one of the assoc. editors in chief of JOSS. perhaps there is some way of receiving feedback? maybe email me to [email protected] |
Thanks @daniel-acuna for offering to do this. The JOSS review process seems super straightforward and transparent: one of the reasons why we chose the venue. @jasmainak will be in touch! |
Yes, my name and affiliation is correct as written. Thanks!
…On Sat, Nov 9, 2019 at 2:05 PM Pavan Ramkumar ***@***.***> wrote:
Thanks @daniel-acuna <https://github.com/daniel-acuna> for offering to do
this. The JOSS review process seems super straightforward and transparent:
one of the reasons why we chose the venue. @jasmainak
<https://github.com/jasmainak> will be in touch!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#313?email_source=notifications&email_token=ABZ7OB2OPVBR523FQMC72ZTQS4JYLA5CNFSM4IYD6K42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDUOAUI#issuecomment-552132689>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABZ7OB6ROX5OXD54U4NGBQTQS4JYLANCNFSM4IYD6K4Q>
.
|
Name and affiliation are good for me. Thanks for including me! Not sure why I'm not showing up in the contributor graph. Perhaps the e-mail I used for that was my undergraduate university e-mail (which I don't have access to anymore) |
Thanks! I would like to be part of this. My name and affiliation look all right. |
Verifying that my name and affiliation is correct as written. Thanks! |
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.
Felt a bit guilty about having my name on something where I haven't contributed, so I added some quick comments. Hope it's helpful. I also hope to look more at the code in the future. Haven't had need to use GLMs in a while. Great work. 👍
paper/paper.md
Outdated
| Group lasso | x | | | x | | | x | | ||
| Tikhonov | x | | | | | | | | ||
|
||
Pyglmnet implements the same algorithm described in [Friedman, J., Hastie, T., & Tibshirani, R. (2010)](https://core.ac.uk/download/files/153/6287975.pdf>) and the accompanying widely popular R package [glmnet]. |
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.
"...implements the algorithm described in..." or "...implements the same algorithm as described in..."
paper/paper.md
Outdated
important to regularize the model to avoid overfitting the data. | ||
Regularization works by adding penalty terms that penalize the model parameters in | ||
a variety of different ways. This can be used to incorporate prior knowledge | ||
about the parameters in a structured form. In Pyglmnet, we offer |
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.
Pyglmnet -> pyglmnet
comment from @DreHar
Thank you for including me. My name and affiliation are right. |
* adding email * Update .mailmap Co-Authored-By: Mainak Jas <[email protected]>
Thanks for including me for the paper. I verified that my name and affiliation are correct. |
Thanks for the update, will be happy to represent the Loyola University Chicago commits. |
@the872 thanks! any idea how we can reach out to @marquesVF -- pending confirmation needed from him |
Hi guys! Thanks for reaching me. That's correct. I was at Loyola. I would only suggest to use my full name as |
closes #297
Hi everyone,
@pavanramkumar and I created a first draft of our paper for submission to JOSS. If you want to be listed as an author, please read the draft and leave comments. More importantly:
pyglmnet
The guidelines for JOSS submission are here.
Compiling the paper
If you want to compile the paper
- go to: https://whedon.theoj.org/
- in the paper repository address, enter: https://github.com/jasmainak/pyglmnet/
- in the custom branch, enter: paper
Compile!
Tagging all contributors (check mark if we got a response)