Skip to content
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

feat: Add section heading information and corresponding authors #35

Merged
merged 2 commits into from
Dec 21, 2020

Conversation

bnachman
Copy link
Collaborator

Hello,

@matthewfeickert started this issue some time ago and I wanted to revisit it. This request is significant and so I have made it a MR instead of automatically updating it (as I have been for the latest papers). Here is the proposal:

  1. Each heading has a small bit of explanation. This appears in the .tex and .pdf versions, but not the markdown (but only because I did not have enough time to make that happen - we can revisit that in the future).
  2. This version will be posted to arXiv. This will improve visibility and linking (e.g. it will also automatically be on Inspire). I would propose that we post new versions to arXiv periodically (perhaps ~1-2 times per year?)
  3. I have listed the author in the .tex and .pdf as "Matthew Feickert and Benjamin Nachman, for the Inter-Experimental LHC Machine Learning Working Group". I don't feel strongly about authorship, but we have to list something in the arXiv post and so feedback is most welcome. We have put some non-trivial amount of time into maintaining this document so perhaps this version is justified.

I have not updated the papers for the week yet in order to reduce any issues for this merge request - once this is approved (or not), I will update the references for the week.

Sincerely,
Ben

@matthewfeickert matthewfeickert added the enhancement New feature or request label Dec 21, 2020
@matthewfeickert matthewfeickert changed the title new structure proposal feat: Add section heading information and corresponding authors Dec 21, 2020
Copy link
Member

@matthewfeickert matthewfeickert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks very much for this excellent PR, @bnachman.

  1. Each heading has a small bit of explanation. This appears in the .tex and .pdf versions, but not the markdown (but only because I did not have enough time to make that happen - we can revisit that in the future).

This is great. Thanks very much for this nice addition, Ben. I'll make an Issue to track/follow up on porting to Markdown later.

  1. This version will be posted to arXiv. This will improve visibility and linking (e.g. it will also automatically be on Inspire). I would propose that we post new versions to arXiv periodically (perhaps ~1-2 times per year?)

Agreed that this is a great way to spread the idea of the review. I think that biannually seems reasonable and if there is a need we can always upload on a more frequent basis as well.

  1. I have listed the author in the .tex and .pdf as "Matthew Feickert and Benjamin Nachman, for the Inter-Experimental LHC Machine Learning Working Group". I don't feel strongly about authorship, but we have to list something in the arXiv post and so feedback is most welcome. We have put some non-trivial amount of time into maintaining this document so perhaps this version is justified.

I think this seems reasonable and, as you point out about arXiv, we can essentially be the corresponding authors for the community to arXiv, which I think the

for the Inter-Experimental LHC Machine Learning Working Group

makes clear.

This all LGTM so I'm very happy to approve this. I'll open up Issues and we can iterate on submit the source to arXiv. :shipit:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants