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

Custom author order (IEEE template) #263

Merged
merged 5 commits into from
Dec 20, 2019
Merged

Custom author order (IEEE template) #263

merged 5 commits into from
Dec 20, 2019

Conversation

DunLug
Copy link
Contributor

@DunLug DunLug commented Dec 12, 2019

Hi, I've done a modification of the IEEE template to allow custom author order.
My thesis director told me that authors should be ordered by importance, but the template was designed to group authors by affiliations.
I add a affiliation.author field to allow breaking order. This shouldn't break compatibility with articles that uses older format.

This is my first pull request on a public repository, so I would be pleased to receive any comment to improve the process next time.

DunLug and others added 3 commits December 12, 2019 11:56
The new yaml field is as follow :
affiliation:
  author:
    - name: First
      mark: 1
    - name: Second
      mark: 2
    - name: Third
      mark: 1
  institution:
    - name: Institution 1
      mark: 1
    - name: Institution 2
      mark: 2

This allows to separate authors of same affiliation while ordering them
by importance.
Copy link
Member

@yihui yihui left a comment

Choose a reason for hiding this comment

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

The code looks good. We just need an example: #265 (review) Thank you!

Copy link
Member

@yihui yihui left a comment

Choose a reason for hiding this comment

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

Thanks!

@yihui yihui merged commit fd09811 into rstudio:master Dec 20, 2019
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants