Closed odow closed 1 year ago
Hi @odow!
Thanks for the rapid feedback, I agree with your comments, particularly that an immediate visual could help users understand the package,
Here's a first list to address your comments. In the Docs:
In the paper:
BlackBoxOptim.jl
and other suitable packages we can find. Regarding the length, I've seen JOSS papers range from 1 paragraph to 6 pages, so I'm not sure if i necessarily agree that the length is extreme, but I know cutting can improve focus and conciseness. When you mention "shrinking", would you prefer less verbose language, or do you have specific pieces you feel readers would not miss being cut.
I'd also move the statement of need to after the summary and before the "Features" section.
i necessarily agree that the length is extreme
I don't know if I said extreme, but it's heading towards the upper end, especially if you add an example with a figure.
When you mention "shrinking", would you prefer less verbose language, or do you have specific pieces you feel readers would not miss being cut.
I thought lines 46-61 could be cut in favor of a sentence like "We also implement a number of features recently described in the literature to improve the robustness and flexibility of the ensemble algorithms (cites...)." But perhaps add the example etc, the see how the length is before cutting.
Thanks for the update and clarification. We will start working on this soon.
Hi @odow Thank you for your patience, our changes:
The latest paper.pdf
Looks good. I wonder if it's still a little long, but I'll leave that to the editors if they complain.
Hi all, I'm reviewing the JOSS paper: https://github.com/openjournals/joss-reviews/issues/4869#issuecomment-1286002558
A couple of things jumped out at me, going off the JOSS checklist:
A statement of need The docs start with
and then it goes into academic literature review. I think you could start with a more generic statement of need that highlighted the derivative free optimization and its applicability for fitting parameters to expensive black-box simulation data.
Example usage The doc order could change. Currently, the examples are right at the bottom, and things like the contributing guide are higher. Provide a simple example as early as possible. Compare the BayesianOptimization intro examples: https://github.com/fmfn/BayesianOptimization#basic-tour-of-the-bayesian-optimization-package
The paper is also quite long. I'd be in favor of shrinking the "features" section in favor of a small code example that demonstrated the package.
Thoughts?