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

Minor fixes JIB cover letter #193

Merged
merged 1 commit into from
Jul 31, 2021
Merged

Minor fixes JIB cover letter #193

merged 1 commit into from
Jul 31, 2021

Conversation

matthiaskoenig
Copy link
Collaborator

  • fix comma behind authors
  • fix order of affilitations
  • minor text updates

* fix comma behind authors
* fix order of affilitations
* minor text updates
@matthiaskoenig
Copy link
Collaborator Author

@luciansmith Here my proposed changes. All minor things. I don't have access to a latex system to compile the pdf at the moment so update of the PDF is missing from this pull request (also could not check if I broke something ;).

@matthiaskoenig matthiaskoenig changed the title Minor fixes Minor fixes JIB cover letter Jul 31, 2021
@luciansmith luciansmith merged commit ebbdc9d into master Jul 31, 2021
@luciansmith luciansmith deleted the matthiaskoenig-jib-1 branch July 31, 2021 21:46
@luciansmith
Copy link
Contributor

Hmm, do we want to do anything about author order here? Making it alphabetical is usually a reasonable way to say 'everyone basically contributed equally'.

@matthiaskoenig
Copy link
Collaborator Author

matthiaskoenig commented Jul 31, 2021 via email

@matthiaskoenig
Copy link
Collaborator Author

matthiaskoenig commented Jul 31, 2021 via email

@luciansmith
Copy link
Contributor

Now I seem to recall some other discussion in the past about authorship--I'm fine leaving the list as-is.

That said, I will happily play videogames against any of you if you want my spot ;-)

@agarny
Copy link
Contributor

agarny commented Aug 1, 2021

My contribution has been somewhat minimal compared with that of others, so happy to go with whatever you guys decide when it comes to authorship.

This being said, I feel like both @matthiaskoenig and @luciansmith have, overall, been the main driving forces behind L1V4. So, personally, I would be more than happy for @luciansmith to be first author and @matthiaskoenig to be last (much better, in my view, to be last if you are thinking about promotion @matthiaskoenig?). As for the rest of us, I guess an alphabetical order would make sense, although I might put @jonrkarr second to acknowledge his comprehensive contribution since he has been on board?

Anyway, just my 2 cents.

@jonrkarr
Copy link
Contributor

jonrkarr commented Aug 2, 2021

The author list also seems random to me.

@agarny suggestions seems reasonable. I would add two suggestions

  • Lucian and Matthias have "*" to indicate equal contribution
  • Since we're each doing this work ourselves, rather than supervising others, I would suggest Lucian and Matthias be the first two authors.

@matthiaskoenig
Copy link
Collaborator Author

matthiaskoenig commented Aug 2, 2021 via email

@luciansmith
Copy link
Contributor

Merging all of the above, I went ahead and put myself as first author, Matthias as last, and everyone else alphabetical. That way we don't have to add a note, and things seem reasonable. Thanks, everyone!

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

Successfully merging this pull request may close these issues.

4 participants