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

[PRE REVIEW]: SWMManywhere: Synthesise Urban Drainage Network Models Anywhere in the World #7438

Closed
editorialbot opened this issue Nov 7, 2024 · 43 comments
Assignees
Labels
pre-review Python Track: 6 (ESE) Earth Sciences and Ecology waitlisted Submissions in the JOSS backlog due to reduced service mode.

Comments

@editorialbot
Copy link
Collaborator

editorialbot commented Nov 7, 2024

Submitting author: @barneydobson (Barnaby Dobson)
Repository: https://github.com/ImperialCollegeLondon/SWMManywhere
Branch with paper.md (empty if default branch):
Version: 0.1.10
Editor: @mengqi-z
Reviewers: @meghnathomas, @cbuahin
Managing EiC: Kristen Thyng

Status

status

Status badge code:

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

Author instructions

Thanks for submitting your paper to JOSS @barneydobson. Currently, there isn't a JOSS editor assigned to your paper.

@barneydobson if you have any suggestions for potential reviewers then please mention them here in this thread (without tagging them with an @). You can search the list of people that have already agreed to review and may be suitable for this submission.

Editor instructions

The JOSS submission bot @editorialbot is here to help you find and assign reviewers and start the main review. To find out what @editorialbot can do for you type:

@editorialbot commands
@editorialbot editorialbot added pre-review Track: 6 (ESE) Earth Sciences and Ecology labels Nov 7, 2024
@editorialbot
Copy link
Collaborator Author

Hello human, I'm @editorialbot, a robot that can help you with some common editorial tasks.

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

@editorialbot commands

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

@editorialbot generate pdf

@editorialbot
Copy link
Collaborator Author

Software report:

github.com/AlDanial/cloc v 1.90  T=0.09 s (1041.1 files/s, 170675.5 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Python                          49           2066           3035           6395
XML                              1              0             52           1534
Markdown                        24            291              0           1070
YAML                            13             43             16            690
TeX                              1             12              2            322
TOML                             1             14              0            151
JSON                             5              1              0             28
CSS                              1              2              0             13
Bourne Shell                     1              0              0              1
-------------------------------------------------------------------------------
SUM:                            96           2429           3105          10204
-------------------------------------------------------------------------------

Commit count by author:

   600	Dobson
   279	barneydobson
    46	dependabot[bot]
    34	Diego Alonso Alvarez
    24	pre-commit-ci[bot]
    13	Tom Bland
     5	Daniel Cummins
     3	Barnaby Dobson
     2	James Paul Turner
     1	AadiUJ
     1	Adrian D'Alessandro
     1	Dan Cummins
     1	Diego Alonso Álvarez

@editorialbot
Copy link
Collaborator Author

Paper file info:

📄 Wordcount for paper.md is 854

✅ The paper includes a Statement of need section

@editorialbot
Copy link
Collaborator Author

License info:

✅ License found: BSD 3-Clause "New" or "Revised" License (Valid open source OSI approved license)

@editorialbot
Copy link
Collaborator Author

⚠️ An error happened when generating the pdf. Problem with ORCID (https://orcid.org/0000-0002-0149-4124) for Barnaby Dobson. ORCID looks to be the wrong length.

@editorialbot
Copy link
Collaborator Author

Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

✅ OK DOIs

- 10.5194/isprs-archives-XLI-B4-125-2016 is OK
- 10.1016/j.compenvurbsys.2017.05.004 is OK

🟡 SKIP DOIs

- No DOI given, and none found for title: Modelling transitions in urban water systems
- No DOI given, and none found for title: Google-Microsoft Open Buildings
- No DOI given, and none found for title: Planet dump retrieved from https://planet.osm.org 
- No DOI given, and none found for title: Overture Maps Foundation

❌ MISSING DOIs

- 10.1007/s12665-011-1146-1 may be a valid DOI for title: Sewer model development under minimum data require...
- 10.1016/j.compenvurbsys.2019.101370 may be a valid DOI for title: Automatic mapping of urban wastewater networks bas...
- 10.1029/2021wr030778 may be a valid DOI for title: A reduced complexity model with graph partitioning...
- 10.1016/j.jhydrol.2018.11.005 may be a valid DOI for title: Identifying fit-for-purpose lumped surrogate model...
- 10.5194/hess-26-4279-2022 may be a valid DOI for title: An algorithm for deriving the topology of belowgro...
- 10.3390/w15010046 may be a valid DOI for title: An evaluation framework for urban pluvial flooding...
- 10.31223/x5gt5x may be a valid DOI for title: SWMManywhere: A global-scale workflow for generati...
- 10.3389/fenvs.2021.626634 may be a valid DOI for title: Data reduced method for cost comparison of wastewa...

❌ INVALID DOIs

- None

@kthyng
Copy link

kthyng commented Nov 7, 2024

👋 @barneydobson - note that your paper does not compile. Please follow the example paper and note that you can click on the error above to find out more about it. Then use the command @editorialbot check references to check again, and the command @editorialbot generate pdf after making changes to the .md file to make a new PDF. editorialbot commands need to be the first entry in a new comment.

@barneydobson
Copy link

@editorialbot check references

Thanks @kthyng - TIL mendeley bibtex export doesn't include DOI 🤦‍♂️

@editorialbot
Copy link
Collaborator Author

Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

✅ OK DOIs

- 10.1007/s12665-011-1146-1 is OK
- 10.1016/j.compenvurbsys.2019.101370 is OK
- 10.1029/2021wr030778 is OK
- 10.1016/j.watres.2017.09.039 is OK
- 10.1016/j.jhydrol.2018.11.005 is OK
- 10.5194/hess-26-4279-2022 is OK
- 10.3390/w15010046 is OK
- 10.31223/X5GT5X is OK
- 10.3389/fenvs.2021.626634 is OK
- 10.5194/isprs-archives-XLI-B4-125-2016 is OK
- 10.1016/j.compenvurbsys.2017.05.004 is OK

🟡 SKIP DOIs

- No DOI given, and none found for title: Google-Microsoft Open Buildings
- No DOI given, and none found for title: Planet dump retrieved from https://planet.osm.org 
- No DOI given, and none found for title: Overture Maps Foundation

❌ MISSING DOIs

- None

❌ INVALID DOIs

- None

@barneydobson
Copy link

@editorialbot generate pdf

@editorialbot
Copy link
Collaborator Author

👉📄 Download article proof 📄 View article proof on GitHub 📄 👈

@editorialbot
Copy link
Collaborator Author

Five most similar historical JOSS papers:

PySWMM: The Python Interface to Stormwater Management Model (SWMM)
Submitting author: @katmratliff
Handling editor: @kbarnhart (Retired)
Reviewers: @nickrsan, @jrkasprzyk
Similarity score: 0.7639

VisWaterNet: A Python package for visualization of water distribution networks
Submitting author: @meghnathomas
Handling editor: @crvernon (Active)
Reviewers: @erexer, @thurber
Similarity score: 0.7314

NeuralHydrology --- A Python library for Deep Learning research in hydrology
Submitting author: @kratzert
Handling editor: @elbeejay (Active)
Reviewers: @ammilten, @chuckaustin, @jhamman
Similarity score: 0.7152

pyflowline: a mesh-independent river network generator for hydrologic models
Submitting author: @changliao1025
Handling editor: @observingClouds (Active)
Reviewers: @smchartrand, @andres-patrignani
Similarity score: 0.7145

HyRiver: Hydroclimate Data Retriever
Submitting author: @cheginit
Handling editor: @kthyng (Active)
Reviewers: @raoulcollenteur, @arbennett
Similarity score: 0.6980

⚠️ Note to editors: If these papers look like they might be a good match, click through to the review issue for that paper and invite one or more of the authors before considering asking the reviewers of these papers to review again for JOSS.

@kthyng
Copy link

kthyng commented Nov 8, 2024

@barneydobson Thanks!

Hi @barneydobson and thanks for your submission! I am looking for some specific items to make sure your submission fits our requirements at a high level (not at the more detailed review level) before moving on to finding an editor or putting this on our waitlist if no relevant editors are available. I'll comment over time as I have a chance to go through them:

In the meantime, please take a look at the comments above ⬆️ from the editorialbot to address any DOI, license, or paper issues if you're able (there may not be any), or suggest reviewers. For reviewers, please suggest 5 reviewers from the database listed above or your own (non-conflicted) extended network. Their github handles are most useful to receive but please don't use "@" to reference them since it will prematurely ping them.

@kthyng
Copy link

kthyng commented Nov 22, 2024

Ok everything looks good here!
We have a backlog of submissions so I will add this to our waitlist. Thanks for your patience.

@kthyng kthyng added the waitlisted Submissions in the JOSS backlog due to reduced service mode. label Nov 22, 2024
@kthyng
Copy link

kthyng commented Dec 19, 2024

@elbeejay Can you edit this submission?

@kthyng
Copy link

kthyng commented Dec 19, 2024

@editorialbot invite @elbeejay as editor

@editorialbot
Copy link
Collaborator Author

Invitation to edit this submission sent!

@elbeejay
Copy link
Member

Apologies for the delay @kthyng - I'm going to have intermittent connectivity now through the end of January, so it might be best for me to not take on anything new until after then.

@kthyng
Copy link

kthyng commented Jan 8, 2025

@elbeejay Ok thanks for the heads up, I'll try to find someone else!

@cheginit Can you edit this submission?

@kthyng
Copy link

kthyng commented Jan 8, 2025

@editorialbot invite @cheginit as editor

@editorialbot
Copy link
Collaborator Author

Invitation to edit this submission sent!

@cheginit
Copy link

cheginit commented Jan 8, 2025

@kthyng Sorry, I'm one of the authors of this submission, so I cannot edit this.

@kthyng
Copy link

kthyng commented Jan 8, 2025

Duh, sorry about that. I saw that earlier when I was looking at this submission in December but wasn't being careful enough this time, and I had forgotten!

@cheginit
Copy link

cheginit commented Jan 8, 2025

Sure, no problem.

@kthyng
Copy link

kthyng commented Jan 14, 2025

@editorialbot invite @mengqi-z as editor

@editorialbot
Copy link
Collaborator Author

Invitation to edit this submission sent!

@kthyng
Copy link

kthyng commented Jan 14, 2025

@mengqi-z Would you be able to edit this submission?

@mengqi-z
Copy link

@kthyng - Yep, I can take on this one.

@mengqi-z
Copy link

@editorialbot assign me as editor

@editorialbot
Copy link
Collaborator Author

Assigned! @mengqi-z is now the editor

@mengqi-z
Copy link

👋 @katmratliff @meghnathomas - Would any of you be available to review a submission for JOSS? Given your background, I believe you'd be a great fit to review this paper.

JOSS uses a checklist-driven review process, which you can find here: https://joss.readthedocs.io/en/latest/review_criteria.html

If you're unable to review, any recommendations for other potential reviewers with relevant expertise would also be greatly appreciated. Thank you!

@mengqi-z mengqi-z reopened this Jan 24, 2025
@meghnathomas
Copy link

Yes I would be happy to!

@mengqi-z
Copy link

@editorialbot add @meghnathomas as reviewer

@editorialbot
Copy link
Collaborator Author

@meghnathomas added to the reviewers list!

@katmratliff
Copy link

@mengqi-z apologies that I am not able to review. I would recommend @cbuahin, if he has bandwidth!

@mengqi-z
Copy link

@katmratliff - Totally understand, thanks for your recommendation!

@mengqi-z
Copy link

@cbuahin - Would you be interested in and available to review this JOSS submission? We carry out our checklist-driven reviews here in GitHub issues and follow these guidelines: https://joss.readthedocs.io/en/latest/review_criteria.html

@cbuahin
Copy link

cbuahin commented Jan 27, 2025

Sure thing @mengqi-z. I am happy to review. However, I have very little bandwidth at the moment and cannot turn this review back around until early March. Let me know If that is acceptable.

@mengqi-z
Copy link

@cbuahin - Thanks for your reply. That would be totally fine! I will set a reminder for you in about 4 weeks.

@mengqi-z
Copy link

@editorialbot add @cbuahin as reviewer

@editorialbot
Copy link
Collaborator Author

@cbuahin added to the reviewers list!

@mengqi-z
Copy link

@editorialbot start review

@editorialbot
Copy link
Collaborator Author

OK, I've started the review over in #7729.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pre-review Python Track: 6 (ESE) Earth Sciences and Ecology waitlisted Submissions in the JOSS backlog due to reduced service mode.
Projects
None yet
Development

No branches or pull requests

9 participants