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

Add affiliation and ORCID iD #8

Merged
merged 1 commit into from
Nov 9, 2022
Merged

Add affiliation and ORCID iD #8

merged 1 commit into from
Nov 9, 2022

Conversation

eddelbuettel
Copy link
Contributor

FIY the one for @coatless looks weird to me in the sense that I would use a plural of 'departments' but now that he is an employee of the former maybe we can standardize and (hi there) use one affiliation for both of us?

@rcurtin
Copy link
Member

rcurtin commented Nov 9, 2022

Ohh, I did a bad job. I thought the extra s was a typo, but I see, it is two separate departments. I should have trusted. 😄 I'll change it back to include the extra s, and @coatless can decide what to do from there. :)

@rcurtin rcurtin merged commit 4ea4f79 into mlpack:main Nov 9, 2022
@coatless
Copy link
Contributor

coatless commented Nov 9, 2022

Technically, I was affiliated with both departments when the work was done. Thus, I think I need to keep it.

@eddelbuettel
Copy link
Contributor Author

Whatever works. I usually take the date of the publication as relevant but it doesn't matter either way. The accidental typo was fixed. If you prefer Depts of Stat + I'tics you got it. Just wanted to give a double heads-up.

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.

3 participants