-
Notifications
You must be signed in to change notification settings - Fork 1
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
Swedish radars sehem, sehud, seoer, seosu #79
Comments
Ok, I have no idea what is going on. Looks in the repository like they might be |
|
It's actually probably the other way around, |
Got this list of new(old) codes for the Swedish radars from Günther: ANG (ANG) |
Aha, that is helpful! 👏
|
I am very helpful really.
|
🙃
|
Documented at #53. I notice that that we are not getting data yet under the new code for 4 of these changes (including |
You might want to compare the lat,lon metadata of the new and old files. It could be that the physical locations of the radars are not exactly the same (different building in the same city), and that therefore they got a new code. |
I will test, compare and report back. |
Ok, here are the results: Coordinatestwo lines = pair of similar radars
Code: library(bioRad)
library(tidyverse)
metadata <- tibble()
# Change code below to select another radar
radar = "sevax"
date_min = "2017-09-01"
date_max = "2017-09-30"
directory = "my_data"
# Run code below to add radar to metadata
download_vpfiles(radar = radar, date_min = date_min, date_max = date_max, directory = directory)
vp_selection <- select_vpfiles(radar = radar, date_min = date_min, date_max = date_max, directory = directory)
vp <- read_vpfiles(vp_selection[1])
new_row <- tibble(
radar = vp$radar,
lat = vp$attributes$where$lat,
long = vp$attributes$where$lon
)
metadata <- bind_rows(metadata, new_row) Distances
Code: library(sf)
sf_metadata <- st_as_sf(metadata, coords = c("long", "lat"), crs = 4326)
st_distance(sf_metadata[9,], sf_metadata[10,]) ConclusionThe radars have different coordinates, but their distance are almost negligible. The biggest difference is between sease and sehem, which is 17m. It still looks like the exact same spot in the forest though: I think we can safely move the old files under the new code directory? |
Great comparison. I still suggest that someone checks directly with the Swedish meteorological Institute to be sure that something hasn’t changed. Some countries are busy replacing radars.
|
It looks good to me. I can check with Günther again, but not really sure what more we want to know? Seems like for our purposes we can just move the files (except for future sebaa, as discussed above). |
Cecilia, if you could just check with Gunther that there haven’t been changes to the radars and just changes to the meta data, that would be more than enough.
From: Cecilia Nilsson [mailto:notifications@github.com]
Sent: Thursday, 11 April 2019 10:28 PM
To: enram/dmp <dmp@noreply.github.com>
Cc: Shamoun-Baranes, Judy <J.Z.Shamoun-Baranes@uva.nl>; Comment <comment@noreply.github.com>
Subject: Re: [enram/dmp] Swedish radars sehem, sehud, seoer, seosu (#5)
It looks good to me. I can check with Günther again, but not really sure what more we want to know? Seems like for our purposes we can just move the files (except for future sebaa, as discussed above).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<https://github.com/enram/dmp/issues/5#issuecomment-482298900>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AHVIzGE5EJZqtyM7PkAPT_EChx_hWTnZks5vf5rXgaJpZM4ckCtS>.
|
Ok, I’ll double check with him so we can be completely sure 👍 |
I sent this summary to Günther a while ago just to make sure we haven't misunderstood anything, waiting for him to reply.
|
Here are Günthers comments (in my translation):
He also said: "Right now only our OPERA single-pol sweeps are being used to create bird profiles (three elevations for upgraded sites and ten for non-upgraded sites, ARL and KIR, but we will change to dual-pol sweeps (10 elevations) as soon as possible. BAA will be the first." |
Thanks @CeciliaNilsson709. Did Gunther also explain why the name changed even when the upgraded radar stayed at the same location? |
Nope. Maybe they just wanted to change it up a bit 💃🤷♀️? Seriously though, he didn't say explicitly, but it seems to be connected "upgrades", which I assume is dual-pol upgrades (but don't know why only some changed). |
Ok, I now know HOW to change this on the S3 bucket. I'll create a spreadsheet to track what needs to be done and what has been done already. 👌 |
All code changes are in effect, except:
It is still submitting under Many of the new codes are not in OPERA DB. |
sekir is still submitting under that code, and not sekrn. As that is the only remaining thing and it is documented in the wiki (https://github.com/enram/data-repository/wiki/se), I will close this issue |
Cf. enram/registry#4, we are getting data from the Swedish radars
sehem
andseoer
:http://enram.github.io/data-repository/?prefix=se/hem/ : 2017, 2018, 2019
http://enram.github.io/data-repository/?prefix=se/oer/ : 2017, 2018, 2019
But these radars are not listed by OPERA: https://github.com/enram/dmp/blob/master/data/OPERA_RADARS_DB.json
@leijnse @CeciliaNilsson709 any information which radars this are and why they are missing in the OPERA database?
The text was updated successfully, but these errors were encountered: