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 a definition for Researchers #237

Merged
merged 2 commits into from
Feb 9, 2024
Merged

Conversation

CaptainSifff
Copy link
Collaborator

I added a quick definition of researchers. Feel free to comment and expand!

@CaptainSifff CaptainSifff linked an issue Feb 1, 2024 that may be closed by this pull request
@mhagdorn
Copy link
Collaborator

mhagdorn commented Feb 2, 2024

I am a little surprised to see librarians mentioned in the same sentence as researchers. Maybe turning the definition around from people who rarely perform RSE activities to people who use RSE services makes a little more sense. In ITIL speak this would be the customers (those who pay the bills) and the users. So maybe

We refer to researchers as people who are using the services provided by Research Software Engineers. These not only include academics but also people from related fields such as librarians and technicians.

@MakisH
Copy link
Collaborator

MakisH commented Feb 2, 2024

Another point: Wouldn't the following order make more sense?

  1. Software
  2. Researcher
  3. Research Software
  4. Research Software Engineers

@CaptainSifff
Copy link
Collaborator Author

I included the text of @mhagdorn and put it up for discussion.

@CaptainSifff
Copy link
Collaborator Author

@MakisH I'd say you would be right usually, but the current text defines "researchers" as people who are not RSEs, so I need their definition before, no?

@MakisH
Copy link
Collaborator

MakisH commented Feb 2, 2024

@MakisH I'd say you would be right usually, but the current text defines "researchers" as people who are not RSEs, so I need their definition before, no?

True. Then the order is fine.

@CaptainSifff CaptainSifff merged commit e46c9bf into main Feb 9, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Define Researchers?
3 participants