Skip to content

Joss submission - #18

Merged
lmoresi merged 10 commits into
mainfrom
joss-submission
Aug 12, 2025
Merged

Joss submission#18
lmoresi merged 10 commits into
mainfrom
joss-submission

Conversation

@lmoresi

Copy link
Copy Markdown
Member

Reviewer / editor feedback for JOSS review from today:

(See openjournals/joss-reviews#7831 (comment) )

Updated paper, fixed bugs, checked doi's and orcid id

lmoresiand others added 5 commits July 29, 2025 17:12
Re-build the joss-submission branch from main
Add pawsey acknowledgement
update BK affiliation
In response to editor review
In response to editor review

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR addresses reviewer/editor feedback for a JOSS (Journal of Open Source Software) submission by updating the paper, fixing bugs, and verifying DOIs and ORCID IDs.

  • Fixed bibliography references and DOI citations
  • Updated acknowledgments section with additional funding sources
  • Corrected formatting for C++ notation in the text

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

FileDescription
docs/joss-paper/paper.mdUpdated bibliography citations, improved C++ formatting, and enhanced acknowledgments
docs/joss-paper/paper.crossrefUpdated timestamp/batch ID and corrected DOI references
docs/joss-paper/paper.bibRemoved file path annotation and corrected DOI for Zienkiewicz reference

Comment threaddocs/joss-paper/paper.md
@lmoresi

Copy link
Copy Markdown
MemberAuthor

We'll need to do this:

  • merge this branch into main
  • bring any bug fixes from dev into main
  • check the CI works correctly / tests passing etc (fix as required)
  • check binder (main) launches, runs correctly (fix as required)
  • make a test release to check Zenodo linking
  • check zenodo authorship for this release matches JOSS authorship
  • bump the version number (1.0 ?)
  • signal JOSS that we are done

L

@lmoresi

Copy link
Copy Markdown
MemberAuthor
  • updated zenodo author list to match JOSS paper
  • checked zenodo linking is active
  • do a test release to validate this, get the badge/doi, update the README

Fixing metadata errors
Ran this through a json linter !
Ben - Curtin affiliation at this stage seems most appropriate
Add zenodo doi
@lmoresi

Copy link
Copy Markdown
MemberAuthor
  • Checked that the releases are working & automatically publishing to zenodo
  • Updated doi / badge in the README

@lmoresi

Copy link
Copy Markdown
MemberAuthor

Just going to push this to main. Changes are entirely within the joss-paper directory.

(and the zenodo.json file)

@lmoresi
lmoresi merged commit 738c686 into mainAug 12, 2025
1 check passed
@lmoresi

Copy link
Copy Markdown
MemberAuthor

Do not delete the joss-submission branch. We can also make this a tag / release but we should keep the branch for now.

Sign up for freeto 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

@lmoresi@bknight1