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

Change the name order and add the affiliation #3

Merged
merged 2 commits into from
Nov 7, 2022

Conversation

KimSangYeon-DGU
Copy link
Member

@KimSangYeon-DGU KimSangYeon-DGU commented Nov 6, 2022

I changed my name order to Sangyeon Kim to comply with the naming convention in English.

Also, I added my affiliation. Is there any form to follow? Please let me know if I'm wrong.

@KimSangYeon-DGU KimSangYeon-DGU changed the title Change the name order Change the name order and add the affiliation Nov 6, 2022
@rcurtin rcurtin merged commit 2fe8ac7 into mlpack:main Nov 7, 2022
@rcurtin
Copy link
Member

rcurtin commented Nov 7, 2022

Thanks! Sorry I got the order wrong. The affiliation isn't quite in the right format but I fixed it in fde2dae. 👍

@KimSangYeon-DGU
Copy link
Member Author

@rcurtin No problem! Thanks for the correction :)

@rcurtin
Copy link
Member

rcurtin commented Nov 14, 2022

@KimSangYeon-DGU also, do you have an ORCID we can add to the paper? Thanks :)

@KimSangYeon-DGU
Copy link
Member Author

@rcurtin Yes, sorry for the late update! Thanks for letting me know :)

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.

2 participants