Repository navigation
Conversation
mhagdorn
requested changes
Oct 9, 2026
mhagdorn
left a comment
mhagdorn
left a comment
Collaborator
There was a problem hiding this comment.
These look great, thanks for the careful read-through.
| The number of people developing software in academia is continuously rising~\autocite{Hannay2009, Hettrick2015}. | ||
| Research software engineering encompasses creating, adapting or maintaining research software. | ||
| It also includes consultation and training with the aim to enable researchers to some of these tasks themselves~\autocite{goth_foundational_competencies_2025}. | ||
| It also includes consultation and training with the aim to enable researchers to do some of these tasks themselves~\autocite{goth_foundational_competencies_2025}. |
Collaborator
There was a problem hiding this comment.
maybe a little better
Suggested change
| It also includes consultation and training with the aim to enable researchers to do some of these tasks themselves~\autocite{goth_foundational_competencies_2025}. | |
| It also includes consultation and training with the aim to enable researchers to perform some of these tasks themselves~\autocite{goth_foundational_competencies_2025}. |
| It makes economic sense to organise such staff centrally, since not every project has a need for a full-time specialist or can afford one over an extended period of time. | ||
| In academia, this is even more relevant, given the wide spectrum of responsibilities that researchers typically have: funding acquisition, project management, teaching loads, networking, conferences, academic self-governance etc. | ||
| A central RSE unit can support here by offloading the technical overhead from the researcher and provide reusable frameworks. | ||
| A central RSE unit can support here by offloading the technical overhead from the researcher and providing reusable frameworks. |
Collaborator
There was a problem hiding this comment.
add missing word
Suggested change
| A central RSE unit can support here by offloading the technical overhead from the researcher and providing reusable frameworks. | |
| A central RSE unit can support here by offloading the technical overhead from the researcher and by providing reusable frameworks. |
| These RSEs typically work isolated from similar RSEs in other groups within the same institution. | ||
| The central RSE unit together with the embedded RSEs form a hub and spokes network. | ||
| These RSEs typically work in isolation from similar RSEs in other groups within the same institution. | ||
| The central RSE unit together with the embedded RSEs form a hub-and-spoke network. |
Collaborator
There was a problem hiding this comment.
hyphens, fair enough, but I think spokes need to be plural as there are many not just a single one
Suggested change
| The central RSE unit together with the embedded RSEs form a hub-and-spoke network. | |
| The central RSE unit together with the embedded RSEs form a hub-and-spokes network. |
| A number of successful RSE units have already been established in Germany and many more exist in other countries, especially the UK and the US\@. | ||
| In order to understand the service portfolio of these existing RSE units, we conducted a small survey that received a total of twelve responses from Germany, the UK and the US\@. | ||
| This very small sample size does of course not allow any quanitative analysis about RSE departments. | ||
| This very small sample size of course does not allow any quantitative analysis of RSE departments. |
Collaborator
There was a problem hiding this comment.
maybe move it to the start
Suggested change
| This very small sample size of course does not allow any quantitative analysis of RSE departments. | |
| Of course, this very small sample size does not allow any quantitative analysis of RSE departments. |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR is a first attempt to address editorial remarks by reviewer 3, which are likely uncontentious.
It also unifies the spelling to British English, which has been used predominantly throughout the document except for a few rare cases.