search

mlpack / mlpack

mlpack: a fast, header-only C++ machine learning library
https://www.mlpack.org/
Other
5.02k stars 1.6k forks source link

Statement of need section in the software paper missing discussion on related libraries #3371

Closed sandeep-ps closed 1 year ago

sandeep-ps commented 1 year ago

This issue is related to the JOSS paper review https://github.com/openjournals/joss-reviews/issues/5026 and checklist entry: Software paper - A statement of need: Does the paper have a section titled 'Statement of need' that clearly states what problems the software is designed to solve, who the target audience is, and its relation to other work?

This issue is also related to https://github.com/mlpack/mlpack/issues/3368 and https://github.com/openjournals/joss-reviews/issues/5026#issuecomment-1382659473.

The current Statement of need section does not discuss how mlpack relates to other similar machine learning C++ libraries. Please add text explaining this aspect.

Also, please add any discussion on the target audience to the paper's Statement of need section. Thanks.

rcurtin commented 1 year ago

Thanks for pointing this out---we added some text to compare mlpack to some other libraries. I'll comment more in the review issue, since the other reviewer wanted to see that improved too. (I'll also get the bot to rerender the paper for easier inspection of changes.) :+1:

sandeep-ps commented 1 year ago

Thanks. I will close this issue.