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

Support for reading in Delft3D FLOW fields #1228

Closed
wants to merge 2 commits into from

Conversation

VeckoTheGecko
Copy link
Contributor

As mentioned in #1205, it would be good to have native support within parcels for reading in hydrodynamic flow output from Delft3D FLOW.

Dev checklist

  • Throw existing implementation into a draft PR
  • Read, document and link Delft3D output specification, to (a) communicate how the Delft grid output works, and (b) direct a robust solution that adresses a wide category of usecases
  • Clean up implementation to be more aligned with the parcels API
  • Implement testing with a variety of different Delft3D FLOW outputs from different usecases
  • Implement warnings for usecases outside of implementation (eg. using curvilinear/3D grids if they aren't supported)
  • Create tutorial .ipynb file to help new users

Timeline

I am happy to continue working on the dev checklist in mid-November 😄

vesnaber added a commit that referenced this pull request Mar 10, 2023
method partially adapted from branch in #1228
@VeckoTheGecko
Copy link
Contributor Author

Superceded by #1331

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.

1 participant