Skip to content

Fm/task/unst 9773 meteo reader#746

Open
FlorisBuwaldaDeltares wants to merge 27 commits intomainfrom
fm/task/UNST-9773_meteo_reader
Open

Fm/task/unst 9773 meteo reader#746
FlorisBuwaldaDeltares wants to merge 27 commits intomainfrom
fm/task/UNST-9773_meteo_reader

Conversation

@FlorisBuwaldaDeltares
Copy link
Copy Markdown
Contributor

@FlorisBuwaldaDeltares FlorisBuwaldaDeltares commented Mar 27, 2026

What was done

  • meteo reader renamed to spatial reader
  • split in file read, file validate, file process, enable quantity
  • remove some unnecessary allocation routines
  • new module m_spatial_field with type t_spatial_field_input to hold read input data
  • added validator unit tests
  • max search radius is now properly passed to the ec module
  • max search radius is now reset to default if not user supplied!
  • qext is now a special case (handled very differently from other quantities)

Evidence of the work done

  • Video/figures
    <add video/figures if applicable>
  • Clear from the issue description
  • Not applicable

Tests

  • Tests updated
    <add testcase numbers if applicable, Issue number>
  • Not applicable

Documentation

  • Documentation updated
    <add description of changes if applicable, Issue number>
  • Not applicable

Issue link

@FlorisBuwaldaDeltares FlorisBuwaldaDeltares marked this pull request as ready for review March 27, 2026 16:16
@FlorisBuwaldaDeltares FlorisBuwaldaDeltares force-pushed the fm/task/UNST-9773_meteo_reader branch from 211324c to bcf75d7 Compare April 1, 2026 09:33
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