Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

Remove some subset variables from datasets #156

Closed
kerfoot opened this issue Sep 9, 2022 · 5 comments
Closed

Remove some subset variables from datasets #156

kerfoot opened this issue Sep 9, 2022 · 5 comments
Assignees

Comments

@kerfoot
Copy link
Contributor

kerfoot commented Sep 9, 2022

The following variables should be removed from the ERDDAP dataset subset service:

  1. trajectory
  2. wmo_id
  3. time_uv
  4. lat_uv
  5. lon_uv
  6. u
  7. v

Current subset variables example:

ru30-20220906T1523.subset

There is only 1 trajectory and wmo_id for each data sets, so subsetting them based on these variables doesn't make sense. It also doesn't make sense to subset by depth-averaged currents, which is what the remaining variables do.

The list of variables providing subsetting should be:

  1. profile_id
  2. time
  3. latitude
  4. longitude
@benjwadams
Copy link
Contributor

Created a PR. Limited subset variables can also be tested on the dev ERDDAP server.

@kerfoot
Copy link
Contributor Author

kerfoot commented Sep 14, 2022

merged and closed.

@kerfoot kerfoot closed this as completed Sep 14, 2022
@kerfoot
Copy link
Contributor Author

kerfoot commented Sep 17, 2022

Reopening...
@benjwadams Existing elements need to have the attribute modified and the catalog needs to be reloaded to remove the bad subset variables.

@kerfoot kerfoot reopened this Sep 17, 2022
@benjwadams
Copy link
Contributor

This has to be applied to production -- will do shortly.

@kerfoot kerfoot self-assigned this Oct 5, 2022
@kerfoot
Copy link
Contributor Author

kerfoot commented Oct 25, 2022

wmo_id and trajectory added to subset variables. Performance is exceptional. Closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants