aimalz / proclam

PRObabilistic CLAssification Metrics for PLAsTiCC
Other
12 stars 7 forks source link

Paper Feedback Round 2: @RickKessler #62

Closed aimalz closed 5 years ago

aimalz commented 5 years ago

I'm seeking volunteers to implement the changes that are not tagged with my name. If you'd like to help, please make a branch, leave a comment on this issue with a link to the branch, check off boxes in this issue as you go, and make a PR as soon as you've gotten through everything you wanted to take on. Thank you!

ABSTRACT

The sentence has been rephrased. --@aimalz

INTRO

The sentence has been rephrased to clarify that the SN Ibc are contaminants accidentally included as if they were SN Ia, not intentionally used with some unspecified procedure. --@aimalz

A log scaling doesn't actually fix the problem, unfortunately. The changes in horizontal ranges are also necessary, and when both are log-scaled, everything gets compressed into the corner even worse. --@aimalz

To preserve interpretability, I'm not going to impose a linear normalization. However, I added some discussion of the magnitude of differences to address this point. --@aimalz

Sec 5:

Sec 6

This is actually part of the LSST-DESC Publication Policy. There have not been problems with including it in LSST-DESC papers thus far. --@aimalz

juramaga commented 5 years ago

Hi Alex, I'll work in this branch: https://github.com/aimalz/proclam/tree/corrections_RM I will start going down the list.

aimalz commented 5 years ago

@juramaga Were you able to push changes? I don't see any edits in your branch, but you've checked off some boxes on the issue. Thanks!

juramaga commented 5 years ago

Hi,

Was in a conference and had to interrupt the work. Starting again now. Will push changes during the day.

Rafael

On Thu, Sep 6, 2018 at 8:58 AM, Alex Malz notifications@github.com wrote:

@juramaga https://github.com/juramaga Were you able to push changes? I don't see any edits in your branch, but you've checked off some boxes on the issue. Thanks!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/aimalz/proclam/issues/62#issuecomment-418986229, or mute the thread https://github.com/notifications/unsubscribe-auth/AFP5A_1-k5MFQkD25p4Q15iNIClQqx-Sks5uYMeRgaJpZM4WVvP4 .

--

J. Rafael Martinez-Galarza Harvard-Smithsonian Center for Astrophysics 60 Garden Street - MS 66 Cambridge, MA 02138 U. S. A. +1 617 495 7027 jmartine@cfa.harvard.edu

juramaga commented 5 years ago

Hi Alex,

I just pushed some changes to my branch.

Rafael

On Thu, Sep 6, 2018 at 8:58 AM, Alex Malz notifications@github.com wrote:

@juramaga https://github.com/juramaga Were you able to push changes? I don't see any edits in your branch, but you've checked off some boxes on the issue. Thanks!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/aimalz/proclam/issues/62#issuecomment-418986229, or mute the thread https://github.com/notifications/unsubscribe-auth/AFP5A_1-k5MFQkD25p4Q15iNIClQqx-Sks5uYMeRgaJpZM4WVvP4 .

--

J. Rafael Martinez-Galarza Harvard-Smithsonian Center for Astrophysics 60 Garden Street - MS 66 Cambridge, MA 02138 U. S. A. +1 617 495 7027 jmartine@cfa.harvard.edu

reneehlozek commented 5 years ago

Finished addressing RK comments.