Skip to content

Conversation

@j-atkins
Copy link
Collaborator

This PR is for visibility of a draft of the VirtualShip JOSS manuscript, and for the review as co-authors to be organised here.

Feel free to add comments and commits to this PR, or branch off as new PRs for suggested changes.


@VeckoTheGecko @iuryt @surgura you have all made substantial contributions to the code. I have listed you as co-authors, alongside @ammedd and @erikvansebille. In the first place, are you happy to be involved with the manuscript? If so, please also let me know what kind of timeline I can expect to get your feedback!

j-atkins and others added 30 commits December 2, 2025 10:59
@j-atkins
Copy link
Collaborator Author

Thanks @iuryt, this is great! Your changes definitely help a lot with readability.

My only subsequent change is that I prefer the colorbars to match the horizontal extent of the subplots, so I have removed fraction and pad from _add_cbar.

figure1

Base automatically changed from delete-spacetime to main January 5, 2026 13:26
@j-atkins
Copy link
Collaborator Author

j-atkins commented Jan 5, 2026

Hi all, happy New Year!

As a reminder, please do give your approval to this PR when you're happy for me to submit to JOSS ✅

cc @ammedd @iuryt @VeckoTheGecko @surgura

Copy link
Collaborator

@ammedd ammedd left a comment

Choose a reason for hiding this comment

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

Whoops, went ahead and pushed some changes I should have written as comments. Just compare and keep what you like. Looks good!

@j-atkins
Copy link
Collaborator Author

j-atkins commented Jan 6, 2026

Whoops, went ahead and pushed some changes I should have written as comments. Just compare and keep what you like. Looks good!

No worries - the changes look good, thanks!

I have changed @surgura's affiliation back to FI though as I believe they paid for his work, according to @erikvansebille's previous comment.

@erikvansebille
Copy link
Member

I have changed @surgura's affiliation back to FI though as I believe they paid for his work, according to @erikvansebille's previous comment.

Indeed, @surgura was paid through the FI as far as I 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.

6 participants