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

Update gtsm_codec path in deltares_data.yml #1016

Open
2 tasks done
dvertegaal opened this issue Jun 28, 2024 · 0 comments
Open
2 tasks done

Update gtsm_codec path in deltares_data.yml #1016

dvertegaal opened this issue Jun 28, 2024 · 0 comments
Labels
Bug Something isn't working Needs refinement issue still needs refinement

Comments

@dvertegaal
Copy link

HydroMT version checks

  • I have checked that this issue has not already been reported.
  • I have checked that this bug exists on the latest version of HydroMT.

Reproducible Example

The current piece of code in the Deltares data catalog for gtsm_codec_reanalysis_{freq}_v1 from: https://raw.githubusercontent.com/Deltares/hydromt/50863d777404a4a91b5f66bf285b3edaa56713b1/data/catalogs/deltares_data.yml

gtsm_codec_reanalysis_{freq}_v1:
  crs: 4326
  data_type: GeoDataset
  driver: netcdf
  kwargs:
    chunks:
      stations: 10
      time: -1
  meta:
    category: ocean
    paper_doi: 10.3389/fmars.2020.00263
    paper_ref: Muis at al (2020)
    source_license: https://cds.climate.copernicus.eu/api/v2/terms/static/licence-to-use-copernicus-products.pdf
    source_url: https://doi.org/10.24381/cds.8c59054f
    source_version: v1
  path: p:/11205028-c3s_435/01_data/01_Timeseries/timeseries2/{variable}/reanalysis_{variable}_{freq}_{year}_{month:02d}_v1.nc
  placeholders:
    freq: [10min, hourly, dailymax]
  rename:
    station_x_coordinate: lon
    station_y_coordinate: lat
    stations: index

Current behaviour

Right now the path misses the new location archivedprojects where the project folder containing the data is moved to: p:/11205028-c3s_435/01_data/01_Timeseries/timeseries2/{variable}/reanalysis_{variable}_{freq}_{year}_{month:02d}_v1.nc

Desired behaviour

As the project folder where the data is stored is moved to archivedprojects, the new path for gtsm_codec_reanalysis_{freq}_v1 in the Deltares data catalog should be:
p:/archivedprojects/11205028-c3s_435/01_data/01_Timeseries/timeseries2/{variable}/reanalysis_{variable}_{freq}_{year}_{month:02d}_v1.nc

Additional context

No response

@dvertegaal dvertegaal added Bug Something isn't working Needs refinement issue still needs refinement labels Jun 28, 2024
@dvertegaal dvertegaal changed the title Incorrect path GTSM datasets Deltares data catalog Update gtsm_codec path in deltares_data.yml Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Needs refinement issue still needs refinement
Projects
None yet
Development

No branches or pull requests

1 participant