Skip to content

Conversation

rohanbabbar04
Copy link
Collaborator

@rohanbabbar04 rohanbabbar04 commented Oct 10, 2024

  • Changes in paper.md

@rohanbabbar04
Copy link
Collaborator Author

rohanbabbar04 commented Oct 10, 2024

@mrava87, I’ve made most of the changes I could, and the paper’s word count has been reduced to around ~1150. Could you take a look and provide any suggestions to reduce it further while keeping the essence of the library intact?
Thank You

@rohanbabbar04 rohanbabbar04 requested a review from mrava87 October 10, 2024 14:45
@mrava87
Copy link
Contributor

mrava87 commented Oct 10, 2024

Sure, I'll try to reduce it further. How do i find out how many words I have after I push the changes?

@rohanbabbar04
Copy link
Collaborator Author

Great!!
I see the word count is now close to 960 words which is good. Are you done with the updates?
I will read the paper when you are done and merge this PR

@mrava87
Copy link
Contributor

mrava87 commented Oct 11, 2024

Perfect, yes I am 😀 feel free to re-add back any sentence that may be worth having but we took out to shorten the paper given that we are now 40 words or so below the limit

@rohanbabbar04
Copy link
Collaborator Author

Perfect, yes I am 😀 feel free to re-add back any sentence that may be worth having but we took out to shorten the paper given that we are now 40 words or so below the limit

Great!!,
Let me merge this

@rohanbabbar04 rohanbabbar04 merged commit ee22336 into joss Oct 11, 2024
3 checks passed
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