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

JOSS Paper #45

Open
mengqi-z opened this issue Sep 20, 2024 · 4 comments
Open

JOSS Paper #45

mengqi-z opened this issue Sep 20, 2024 · 4 comments

Comments

@mengqi-z
Copy link

Hi @lilyclements,

I have a few comments on the paper that need to be addressed before we proceed to the next stage:

  1. Lines 52-53 duplicate the content from lines 49-51. I recommend removing lines 52-53 or merging with previous paragraph to avoid redundancy.

  2. Usage Examples. On lines 62 and 67, please prepend carbonr:: to the functions to ensure consistency with the formatting used for other functions in the examples.

  3. References.

    • For MacNeil et al., 2017. Please add the DOI: 10.1016/S2542-5196(17)30162-6
    • For Schmidt and Kulma, 2024. Please update the link to the appropriate DOI: 10.32614/CRAN.package.footprint

Thank you.

This issue is for JOSS paper review openjournals/joss-reviews#6932

lilyclements added a commit that referenced this issue Sep 21, 2024
@lilyclements
Copy link
Collaborator

Thank you @mengqi-z for this. It's very helpful. I have fixed this in PR #46. If you're happy with the changes then let me know and I will close this issue. Thanks.

@mengqi-z
Copy link
Author

Thank you @lilyclements! I have one more thing I realized just now. Could you please add the city, country in the affiliation if possible? Thanks!

@lilyclements
Copy link
Collaborator

@mengqi-z thanks - have done (PR #48)

@mengqi-z
Copy link
Author

Thanks! All looks good now!

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

No branches or pull requests

2 participants