[REVIEW]: SHARPy: A dynamic aeroelastic simulation toolbox for very flexible aircraft and wind turbines #1885

whedon · 2019-11-11T21:49:33Z

Submitting author: @fonsocarre (Alfonso del Carre)
Repository: https://github.com/ImperialCollegeLondon/sharpy
Version: v1.1.0-2
Editor: @labarba
Reviewer: @petebachant, @rafmudaf
Archive: 10.5281/zenodo.3572512

Status

Status badge code:

HTML: <a href="https://joss.theoj.org/papers/f7ccd562160f1a54f64a81e90f5d9af9"><img src="https://joss.theoj.org/papers/f7ccd562160f1a54f64a81e90f5d9af9/status.svg"></a>
Markdown: [![status](https://joss.theoj.org/papers/f7ccd562160f1a54f64a81e90f5d9af9/status.svg)](https://joss.theoj.org/papers/f7ccd562160f1a54f64a81e90f5d9af9)

Reviewers and authors:

Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) by leaving comments in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)

Reviewer instructions & questions

@petebachant & @rafmudaf, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:

Make sure you're logged in to your GitHub account
Be sure to accept the invite at this URL: https://github.com/openjournals/joss-reviews/invitations

The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @labarba know.

✨ Please try and complete your review in the next two weeks ✨

Review checklist for @petebachant

Conflict of interest

I confirm that I have read the JOSS conflict of interest (COI) policy and that: I have no COIs with reviewing this work or that any perceived COIs have been waived by JOSS for the purpose of this review.

Code of Conduct

I confirm that I read and will adhere to the JOSS code of conduct.

General checks

Repository: Is the source code for this software available at the repository url?
License: Does the repository contain a plain-text LICENSE file with the contents of an OSI approved software license?
Contribution and authorship: Has the submitting author (@fonsocarre) made major contributions to the software? Does the full list of paper authors seem appropriate and complete?

Functionality

Installation: Does installation proceed as outlined in the documentation?
Functionality: Have the functional claims of the software been confirmed?
Performance: If there are any performance claims of the software, have they been confirmed? (If there are no claims, please check off this item.)

Documentation

A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?
Installation instructions: Is there a clearly-stated list of dependencies? Ideally these should be handled with an automated package management solution.
Example usage: Do the authors include examples of how to use the software (ideally to solve real-world analysis problems).
Functionality documentation: Is the core functionality of the software documented to a satisfactory level (e.g., API method documentation)?
Automated tests: Are there automated tests or manual steps described so that the functionality of the software can be verified?
Community guidelines: Are there clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support

Software paper

Summary: Has a clear description of the high-level functionality and purpose of the software for a diverse, non-specialist audience been provided?
A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?
State of the field: Do the authors describe how this software compares to other commonly-used packages?
Quality of writing: Is the paper well written (i.e., it does not require editing for structure, language, or writing quality)?
References: Is the list of references complete, and is everything cited appropriately that should be cited (e.g., papers, datasets, software)? Do references in the text use the proper citation syntax?

Review checklist for @rafmudaf

Conflict of interest

I confirm that I have read the JOSS conflict of interest (COI) policy and that: I have no COIs with reviewing this work or that any perceived COIs have been waived by JOSS for the purpose of this review.

Code of Conduct

I confirm that I read and will adhere to the JOSS code of conduct.

General checks

Repository: Is the source code for this software available at the repository url?
License: Does the repository contain a plain-text LICENSE file with the contents of an OSI approved software license?
Contribution and authorship: Has the submitting author (@fonsocarre) made major contributions to the software? Does the full list of paper authors seem appropriate and complete?

Functionality

Installation: Does installation proceed as outlined in the documentation?
Functionality: Have the functional claims of the software been confirmed?
Performance: If there are any performance claims of the software, have they been confirmed? (If there are no claims, please check off this item.)

Documentation

A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?
Installation instructions: Is there a clearly-stated list of dependencies? Ideally these should be handled with an automated package management solution.
Example usage: Do the authors include examples of how to use the software (ideally to solve real-world analysis problems).
Functionality documentation: Is the core functionality of the software documented to a satisfactory level (e.g., API method documentation)?
Automated tests: Are there automated tests or manual steps described so that the functionality of the software can be verified?
Community guidelines: Are there clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software 3) Seek support

Software paper

Summary: Has a clear description of the high-level functionality and purpose of the software for a diverse, non-specialist audience been provided?
A statement of need: Do the authors clearly state what problems the software is designed to solve and who the target audience is?
State of the field: Do the authors describe how this software compares to other commonly-used packages?
Quality of writing: Is the paper well written (i.e., it does not require editing for structure, language, or writing quality)?
References: Is the list of references complete, and is everything cited appropriately that should be cited (e.g., papers, datasets, software)? Do references in the text use the proper citation syntax?

The text was updated successfully, but these errors were encountered:

whedon · 2019-11-11T21:49:36Z

Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @petebachant, @rafmudaf it looks like you're currently assigned to review this paper 🎉.

⭐ Important ⭐

If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews 😿

To fix this do the following two things:

Set yourself as 'Not watching' https://github.com/openjournals/joss-reviews:

You may also like to change your default settings for this watching repositories in your GitHub profile here: https://github.com/settings/notifications

For a list of things I can do to help you, just type:

@whedon commands

For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:

@whedon generate pdf

whedon · 2019-11-11T21:49:37Z

Attempting PDF compilation. Reticulating splines etc...

whedon · 2019-11-11T21:50:07Z

👉 Check article proof 📄 👈

labarba · 2019-11-11T21:51:06Z

👋 @petebachant, @rafmudaf — Thank you for agreeing to review for JOSS! This is where the action happens: work your way through the review checklist, feel free to ask questions or post comments here, and also open issues in the submission repository as needed (mentioning this issue, for a cross-link). Godspeed!

petebachant · 2019-12-02T23:56:28Z

@whedon generate pdf

whedon · 2019-12-02T23:56:30Z

Attempting PDF compilation. Reticulating splines etc...

whedon · 2019-12-02T23:57:08Z

👉 Check article proof 📄 👈

petebachant · 2019-12-06T00:35:15Z

I noticed a small error in one of the sample cases, but was able to confirm the functionality of the software with a different one. I recommend publication.

rafmudaf · 2019-12-08T21:52:44Z

After the latest updates to the build system, I was able to compile on macOS exactly as documented. I also was able to run the example cases and confirm the functionality. Very good work!

labarba · 2019-12-08T22:02:18Z

@petebachant, @rafmudaf —I see that both of you have checked off all the items in the review checklist. Do we have a recommendation to publish, from each of you?

rafmudaf · 2019-12-08T22:03:18Z

Yes, I recommend publishing.

petebachant · 2019-12-08T22:05:28Z

I also recommend publishing.

labarba · 2019-12-08T22:12:02Z

@whedon generate pdf

whedon · 2019-12-08T22:12:04Z

Attempting PDF compilation. Reticulating splines etc...

whedon · 2019-12-08T22:12:34Z

👉 Check article proof 📄 👈

labarba · 2019-12-08T22:26:46Z

@fonsocarre — small typo: ¶4 "These longer and much slender…" >> These longer and more slender...

Also, please use consistent capitalization for your references: normal cap for article titles, camel case for journal or conference title. The first two references need proper capitalization of the conference (scitech forum >> SciTech Forum), and the 7th reference needs the article title in normal cap.

Once you make these small changes, please go ahead and make a new tagged release of your software, and report the version number here, then archive your software repository in Zenodo (or similar) and please report the DOI here.

ngoiz · 2019-12-08T22:34:20Z

Thanks @labarba, @petebachant and @rafmudaf for the review and suggestions!

We will update the JOSS submission with the above, release and share the DOI.

Thanks!

fonsocarre · 2019-12-12T15:37:36Z

@whedon generate pdf

whedon · 2019-12-12T15:37:39Z

Attempting PDF compilation. Reticulating splines etc...

whedon · 2019-12-12T15:38:28Z

PDF failed to compile for issue #1885 with the following error:

[WARNING] Could not convert image '/tmp/tex2pdf.-10d38b739308f138/686c07f142856d364b54fcb193b3058840e96c03.shtml': Cannot load file
Jpeg Invalid marker used
PNG Invalid PNG file, signature broken
Bitmap Invalid Bitmap magic identifier
GIF Invalid Gif signature :

HDR Invalid radiance file signature
Tiff Invalid endian tag value
TGA Invalid bit depth (104)
Error producing PDF.
! LaTeX Error: Cannot determine size of graphic in /tmp/tex2pdf.-10d38b739308f1
38/686c07f142856d364b54fcb193b3058840e96c03.shtml (no BoundingBox).

See the LaTeX manual or LaTeX Companion for explanation.
Type H for immediate help.
...

l.421 ...f142856d364b54fcb193b3058840e96c03.shtml}

Looks like we failed to compile the PDF

rafmudaf · 2019-12-12T17:20:44Z

Likewise, I'm happy to be a part of the process. Keep up the good work!

labarba · 2019-12-12T23:42:45Z

@whedon set v1.1.0-2 as version

whedon · 2019-12-12T23:42:50Z

OK. v1.1.0-2 is the version.

labarba · 2019-12-12T23:43:48Z

@fonsocarre — I'm going to ask you to edit the metadata of the Zenodo deposit (no need to get new version or new DOI) so the title and author list match the JOSS paper.

fonsocarre · 2019-12-13T10:09:08Z

@labarba Done. It looks like it is not instantaneous, but it should show up soon.

labarba · 2019-12-13T13:07:42Z

Something didn't quite work ... can you double check?

fonsocarre · 2019-12-13T14:20:42Z

100% my fault. I saved, but didn't publish.

labarba · 2019-12-13T15:25:23Z

We generally ask for the title of the Zenodo archive to match the title of the JOSS paper, but you didn't make that change. Do you have any strong reasons?

fonsocarre · 2019-12-13T15:38:24Z

I do not. It has been modified now.

labarba · 2019-12-13T19:08:23Z

@whedon set 10.5281/zenodo.3572512 as archive

whedon · 2019-12-13T19:08:27Z

OK. 10.5281/zenodo.3572512 is the archive.

labarba · 2019-12-13T19:08:42Z

@whedon accept

whedon · 2019-12-13T19:08:45Z

Attempting dry run of processing paper acceptance...

whedon · 2019-12-13T19:09:22Z


OK DOIs

- 10.2514/2.2738 is OK
- 10.2514/6.2013-1634 is OK
- 10.1017/s0001924000010952 is OK
- 10.2514/6.2011-1226 is OK
- 10.2514/6.1999-1394 is OK
- 10.2514/1.j051392 is OK
- 10.2514/1.c032464 is OK
- 10.2514/1.j050496 is OK
- 10.2514/6.2013-4745 is OK
- 10.1016/j.paerosci.2012.06.001 is OK
- 10.2514/1.j050513 is OK
- 10.2514/6.2018-1202 is OK
- 10.2514/1.j051697 is OK
- 10.2514/6.2002-1719 is OK
- 10.2514/6.2006-1638 is OK
- 10.2514/1.J052316 is OK
- 10.1016/j.ijsolstr.2011.04.012 is OK
- 10.2514/1.C032477 is OK
- 10.2514/1.47317 is OK
- 10.2514/1.27606 is OK
- 10.2514/1.g000715 is OK
- 10.2514/6.2019-2038 is OK
- 10.2514/1.J051543 is OK
- 10.2514/1.J052136 is OK
- 10.2514/6.2007-6703 is OK
- 10.1002/qj.49709841707 is OK
- https://doi.org/10.1016/j.paerosci.2018.03.006 is OK
- 10.2514/1.J058153 is OK

MISSING DOIs

- https://doi.org/10.2514/1.c032222 may be missing for title: Reduced-Order Modeling of Unsteady Aerodynamics Across Multiple Mach Regimes
- https://doi.org/10.2514/mscitech19 may be missing for title: AIAA SciTech 2019 Forum
- https://doi.org/10.2307/2003941 may be missing for title: A class of methods for solving nonlinear simultaneous equations

INVALID DOIs

- None

whedon · 2019-12-13T19:09:30Z

Check final proof 👉 openjournals/joss-papers#1173

If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#1173, then you can now move forward with accepting the submission by compiling again with the flag deposit=true e.g.

@whedon accept deposit=true

labarba · 2019-12-13T21:00:39Z

@whedon accept deposit=true

whedon · 2019-12-13T21:00:42Z

Doing it live! Attempting automated processing of paper acceptance...

whedon · 2019-12-13T21:05:16Z

🐦🐦🐦 👉 Tweet for this paper 👈 🐦🐦🐦

whedon · 2019-12-13T21:05:17Z

🚨🚨🚨 THIS IS NOT A DRILL, YOU HAVE JUST ACCEPTED A PAPER INTO JOSS! 🚨🚨🚨

Here's what you must now do:

Check final PDF and Crossref metadata that was deposited 👉 Creating pull request for 10.21105.joss.01885 joss-papers#1174
Wait a couple of minutes to verify that the paper DOI resolves https://doi.org/10.21105/joss.01885
If everything looks good, then close this review issue.
Party like you just published a paper! 🎉🌈🦄💃👻🤘

Any issues? notify your editorial technical team...

labarba · 2019-12-13T21:07:27Z

Congratulations, @fonsocarre, your JOSS paper is published! 🚀

Huge thanks to @petebachant, @rafmudaf for reviewing this submission 🙏

whedon · 2019-12-13T21:16:02Z

🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉

If you would like to include a link to your paper from your README use the following code snippets:

Markdown:
[![DOI](https://joss.theoj.org/papers/10.21105/joss.01885/status.svg)](https://doi.org/10.21105/joss.01885)

HTML:
<a style="border-width:0" href="https://doi.org/10.21105/joss.01885">
  <img src="https://joss.theoj.org/papers/10.21105/joss.01885/status.svg" alt="DOI badge" >
</a>

reStructuredText:
.. image:: https://joss.theoj.org/papers/10.21105/joss.01885/status.svg
   :target: https://doi.org/10.21105/joss.01885

This is how it will look in your documentation:

We need your help!

Journal of Open Source Software is a community-run journal and relies upon volunteer effort. If you'd like to support us please consider doing either one (or both) of the the following:

Volunteering to review for us sometime in the future. You can add your name to the reviewer list here: https://joss.theoj.org/reviewer-signup.html
Making a small donation to support our running costs here: https://numfocus.org/donate-to-joss

whedon added the review label Nov 11, 2019

whedon assigned labarba Nov 11, 2019

whedon mentioned this issue Nov 11, 2019

[PRE REVIEW]: SHARPy: A dynamic aeroelastic simulation toolbox for very flexible aircraft and wind turbines #1878

Closed

whedon assigned petebachant and rafmudaf Nov 12, 2019

This was referenced Nov 24, 2019

Change position of --name argument in docs ImperialCollegeLondon/sharpy#35

Merged

Compare to other competing software in JOSS paper ImperialCollegeLondon/sharpy#36

Closed

run_theo_freq.py fails in Docker container ImperialCollegeLondon/sharpy#37

Closed

rafmudaf mentioned this issue Nov 27, 2019

Installation fails on macOS with Intel compiler ImperialCollegeLondon/sharpy#46

Closed

petebachant mentioned this issue Dec 5, 2019

WindTurbine case generation script does not produce sharpy file ImperialCollegeLondon/sharpy#50

Closed

whedon added the accepted label Dec 13, 2019

labarba closed this as completed Dec 13, 2019

whedon added published Papers published in JOSS recommend-accept Papers recommended for acceptance in JOSS. labels Mar 2, 2020

editorialbot mentioned this issue Nov 27, 2023

[PRE REVIEW]: jaxparrow: a Python package solving the cyclogeostrophic balance using a variational formulation #6095

Closed

rafmudaf mentioned this issue Mar 7, 2024

[PRE REVIEW]: open-mastr: A Python Package to Download and Process the German Energy Registry Marktstammdatenregister #6199

Closed

editorialbot mentioned this issue Jul 13, 2024

[PRE REVIEW]: TurboFlow: Meanline Modelling of Turbomachinery #6979

Closed

editorialbot mentioned this issue Sep 26, 2024

[PRE REVIEW]: pyOMA2: a Python module to conduct operational modal analysis #7293

Closed

editorialbot mentioned this issue Nov 18, 2024

[PRE REVIEW]: FAST: A Future Aircraft Sizing Tool for Advanced Aircraft and Propulsion System Design #7497

Open

[REVIEW]: SHARPy: A dynamic aeroelastic simulation toolbox for very flexible aircraft and wind turbines #1885

[REVIEW]: SHARPy: A dynamic aeroelastic simulation toolbox for very flexible aircraft and wind turbines #1885

Comments

whedon commented Nov 11, 2019 • edited Loading

Status

Reviewer instructions & questions

Review checklist for @petebachant

Conflict of interest

Code of Conduct

General checks

Functionality

Documentation

Software paper

Review checklist for @rafmudaf

Conflict of interest

Code of Conduct

General checks

Functionality

Documentation

Software paper

whedon commented Nov 11, 2019

whedon commented Nov 11, 2019

whedon commented Nov 11, 2019

labarba commented Nov 11, 2019

petebachant commented Dec 2, 2019

whedon commented Dec 2, 2019

whedon commented Dec 2, 2019

petebachant commented Dec 6, 2019 • edited Loading

rafmudaf commented Dec 8, 2019

labarba commented Dec 8, 2019

rafmudaf commented Dec 8, 2019

petebachant commented Dec 8, 2019

labarba commented Dec 8, 2019

whedon commented Dec 8, 2019

whedon commented Dec 8, 2019

labarba commented Dec 8, 2019

ngoiz commented Dec 8, 2019

fonsocarre commented Dec 12, 2019

whedon commented Dec 12, 2019

whedon commented Dec 12, 2019

rafmudaf commented Dec 12, 2019

labarba commented Dec 12, 2019

whedon commented Dec 12, 2019

labarba commented Dec 12, 2019

fonsocarre commented Dec 13, 2019

labarba commented Dec 13, 2019

fonsocarre commented Dec 13, 2019

labarba commented Dec 13, 2019

fonsocarre commented Dec 13, 2019

labarba commented Dec 13, 2019

whedon commented Dec 13, 2019

labarba commented Dec 13, 2019

whedon commented Dec 13, 2019

whedon commented Dec 13, 2019

whedon commented Dec 13, 2019

labarba commented Dec 13, 2019

whedon commented Dec 13, 2019

whedon commented Dec 13, 2019

whedon commented Dec 13, 2019

labarba commented Dec 13, 2019

whedon commented Dec 13, 2019

whedon commented Nov 11, 2019 •

edited

Loading

petebachant commented Dec 6, 2019 •

edited

Loading