CMakePP / CMakePPLang

Object-oriented extension to the CMake language.
http://cmakepp.github.io/CMakePPLang/
Apache License 2.0
11 stars 4 forks source link

Joss paper #87

Closed ryanmrichard closed 8 months ago

ryanmrichard commented 1 year ago

This PR should NOT be merged into master.

The purpose of this PR is to prepare a submission to JOSS. The PR itself serves as a way for the team to communicate and collaborate on the paper contents. and, IIRC, JOSS is going to require us to have a PR anyways.

ryanmrichard commented 1 year ago

Some notes on the front matter:

ryanmrichard commented 1 year ago

By other comment is that the code snippets are a bit small in the PDF

ryanmrichard commented 1 year ago

@blakemulnix and @AutonomicPerfectionist we're pretty close to being ready to submit this. So now's your chance to read and comment on the paper before we submit it. The paper is pretty short (only four pages, one page is references and another is code snippets) and is available in a zip file here.

If you do read it, please let us know if you have any comments or suggestions. If you don't want to read it that's fine too. Either way, please let us know that you are okay submitting the paper.

AutonomicPerfectionist commented 1 year ago

@ryanmrichard I read through the paper and it looks good to me. I'm okay with submitting it as-is

ryanmrichard commented 1 year ago

@AutonomicPerfectionist Thanks!

blakemulnix commented 1 year ago

Looks good to me!

ryanmrichard commented 1 year ago

@blakemulnix and @AutonomicPerfectionist do either of you have ORCiD? You don't need to have one, but if you have one I want to make sure it gets on the paper.

blakemulnix commented 1 year ago

I do not have ORCiD.

AutonomicPerfectionist commented 1 year ago

I do not either

ryanmrichard commented 11 months ago

@AutonomicPerfectionist the tests are now failing for this branch. Is it possible we need to update something owing to new CMakeTest changes?

AutonomicPerfectionist commented 11 months ago

@AutonomicPerfectionist the tests are now failing for this branch. Is it possible we need to update something owing to new CMakeTest changes?

I took a look, it seems to be the name of a test is being interpreted as a type, causing the overload checks to fail. I'll look into it further and see what needs to be done

ryanmrichard commented 11 months ago

@AutonomicPerfectionist for now I'm just going to pin the version of CMakeTest back a commit (arguably we should have been pining the version anyways), that'll make the tests pass again. Longer term we need to figure out if we want to allow test names which are recognized as types (I'm inclined to say yes as it arose quite naturally here).

ryanmrichard commented 11 months ago

All proofs are available here. Please let me know as soon as possible (ideally in the next couple of days) if there's anything you want to change.

zachcran commented 8 months ago

@ryanmrichard Should this PR be closed now?