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

Joss paper #313

Open
wants to merge 26 commits into
base: main
Choose a base branch
from
Open

Joss paper #313

wants to merge 26 commits into from

Conversation

barneydobson
Copy link
Collaborator

@barneydobson barneydobson commented Oct 16, 2024

Description

  • Prepare the JOSS paper file.
  • Link the paper as an 'about' section in documentation. (You can view it with mkdocs serve) Now a pdf that is compiled on publish.
  • fix link for coverage

@cheginit this is a super quick draft I thought I'd prepare for a JOSS paper. I've just gone for something similar to the WSIMOD one. I haven't checked in detail at all and definitely need to polish it - but just wanted to check that you're happy with the broad format (i.e., summary = 1 para, statement of need = 1 para literature + 1 para software, outlook/limitations = 1 para). In particular I'm aware the literature para needs some significant improvement!

OK I've actually reduced it and moved the literature into the summary, but I think it works better, LMK what you think (@cheginit )

Fixes #337

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature (non-breaking change which adds functionality)
  • Optimization (non-breaking, back-end change that speeds up the code)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Key checklist

  • All tests pass (eg. python -m pytest)
  • The documentation builds and looks OK (eg. python -m sphinx -b html docs docs/build)
  • Pre-commit hooks run successfully (eg. pre-commit run --all-files)

Further checks

  • Code is commented, particularly in hard-to-understand areas
  • Tests added or an issue has been opened to tackle that in the future. (Indicate issue here: # (issue))

@codecov-commenter
Copy link

codecov-commenter commented Oct 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.79%. Comparing base (fb919bc) to head (65d7740).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #313      +/-   ##
==========================================
+ Coverage   87.77%   87.79%   +0.01%     
==========================================
  Files          22       22              
  Lines        2217     2220       +3     
  Branches      280      280              
==========================================
+ Hits         1946     1949       +3     
  Misses        194      194              
  Partials       77       77              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@barneydobson barneydobson marked this pull request as draft October 16, 2024 14:05
@cheginit
Copy link
Collaborator

The structure is good. Just try to keep it below 1000 words.

@barneydobson barneydobson marked this pull request as ready for review October 22, 2024 16:16

`SWMManywhere` is an open-source Python package designed for the global synthesis of urban drainage networks. SWMManywhere integrates publicly available geospatial data and automates data acquisition and preprocessing, reducing the technical burden on users. The CLI offers a flexible workflow, providing an accessible entry point to using and customising synthesis. Its parameterized design enables detailed sensitivity analyses, allowing users to understand and manage uncertainties inherent in urban drainage modelling [@Dobson2024-dv]. By emphasizing user control, SWMManywhere allows tuning of outputs with parameters to meet local requirements, making it adaptable to a wide range of scenarios. Designed for both researchers and practitioners in urban water management, SWMManywhere responds to the limitations of existing methods by providing an end-to-end, open-source, and customisable solution.

## Outlook
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It feels odd that most of the paper talks about what it does not do, yet, rather than what it does, its unique features or what does better than other approaches. I know you do need to talk about this - and you phrased it very well as research opportunities rather than software limitations - but even so.

I would include a section on Features - or maybe as part of the statement of need - and a picture (maybe a map of the notebook with the calculated swage system) and reduce this section a little bit..

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good suggestions!

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.

network.node_link_data
4 participants