Skip to content

Commit

Permalink
drop andratx station again for cmems-nrt (#1023)
Browse files Browse the repository at this point in the history
* drop andratx station again for cmems-nrt

* updated changelog
  • Loading branch information
veenstrajelmer authored Oct 11, 2024
1 parent 6979268 commit fc9dcf7
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 1 deletion.
8 changes: 7 additions & 1 deletion dfm_tools/observations.py
Original file line number Diff line number Diff line change
Expand Up @@ -252,7 +252,13 @@ def cmems_ssh_read_catalog(source, overwrite=True):
bool_tidegauge = index_history_pd["file_name"].str.contains("/history/TG/")
bool_slev = index_history_pd["parameters"].str.contains("SLEV")
index_history_pd = index_history_pd.loc[bool_tidegauge & bool_slev]


# drop andratx station, lat/lon vary over time in nrt dataset
# TODO: remove this exception when the CMEMS nrt dataset is cleaned up
if source == "cmems-nrt":
bool_moving = index_history_pd["file_name"].str.contains("MO_TS_TG_ANDRATX")
index_history_pd = index_history_pd.loc[~bool_moving]

# generate geom and geodataframe
assert (index_history_pd["geospatial_lon_min"] == index_history_pd["geospatial_lon_max"]).all()
assert (index_history_pd["geospatial_lat_min"] == index_history_pd["geospatial_lat_max"]).all()
Expand Down
6 changes: 6 additions & 0 deletions docs/whats-new.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
# What's new

## UNRELEASED

### Fix
- fixed cmems-nrt insitu again by dropping station with varying coordinates again in [#1023](https://github.com/Deltares/dfm_tools/pull/1023)


## 0.29.0 (2024-09-27)

### Feat
Expand Down

0 comments on commit fc9dcf7

Please sign in to comment.