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

Proofs for JOSS #11

Merged
merged 6 commits into from
Oct 25, 2021
Merged

Proofs for JOSS #11

merged 6 commits into from
Oct 25, 2021

Conversation

Bisaloo
Copy link
Contributor

@Bisaloo Bisaloo commented Oct 25, 2021

Here are a couple of minor changes before final acceptance (JOSS doesn't have an external proof service). I recommend you give it a final read as well to be sure everything is fine before publication.

Preview here: https://res.cloudinary.com/hkvhhndao/image/upload/v1635157383/ndf8jrv9caupcb9jz23g.pdf

It would be amazing it you could also provide the figure as a pdf file instead of png. Don't worry if you can't, it's not such a big deal.

Related to openjournals/joss-reviews#3705.

@ejikeugba
Copy link
Owner

The proofs look good, thanks! I guess the second duplicate paragraph should be dropped.

Okay, then we move on with the png format since reconverting to pdf could reduce the quality of the plot.
I will go through everything once again and then release and archive in zenodo.

@ejikeugba ejikeugba merged commit e1b255b into ejikeugba:master Oct 25, 2021
@Bisaloo Bisaloo deleted the proofs branch October 25, 2021 21:42
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.

2 participants