-
Notifications
You must be signed in to change notification settings - Fork 10
adapt manuscript to new JOSS scope requirements #270
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
Conversation
erikvansebille
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good; just three small comments below
docs/paper/jats/paper.jats
Outdated
| refactoring and test scaffolding. AI-assisted autocompletion tools | ||
| (via GitHub Copilot) were used in the writing of this manuscript. | ||
| Authors carefully reviewed and edited all AI-assisted content and made | ||
| the core desigin decisions.</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| the core desigin decisions.</p> | |
| the core desigin decisions without use of AI.</p> |
|
@surgura I seemingly can't request your review at the moment, unfortunately. If you start participating I may be able to, but otherwise feel free to simply state your approval/react to this comment. |
|
I also don't seem to be allowed to leave my approval so we'll do it manually =) thanks for the effort |
|
All good from my side too |
Following on from my comment in the original JOSS manuscript PR, here is the adapted manuscript, adapted to the new JOSS requirements.
In practice, it required minimal changes because the content largely addressed the new requirements anyway but some heading titles needed to be changed and an AI disclosure statement added.
cc @erikvansebille @ammedd @VeckoTheGecko @iuryt @surgura