Whatever happened with shape names?? #2655
-
|
Why are all shape names missing from the data? |
Beta Was this translation helpful? Give feedback.
Replies: 2 comments 9 replies
-
|
Uh oh - which dataset? I can dig into this later today and try to roll out a quick fix. |
Beta Was this translation helpful? Give feedback.
-
|
Every?? Angola Admin2, Austria Admin2... Those are the only ones I checked, but the field seems to be completely missing. |
Beta Was this translation helpful? Give feedback.
-
|
Let me check this now. I just pushed 5.0, and didn't get any flags, but that may be due to the migration of the build scripts to the local environment. Will ping back here shortly; hopefully can get this fixed tonight (tomorrow AM at the latest). |
Beta Was this translation helpful? Give feedback.
-
|
Just confirming that, yes, shapeName is missing, which is an error. It didn't get flagged because it's technically optional (i.e., we don't have it for every case), but it should still be present in most cases. |
Beta Was this translation helpful? Give feedback.
-
|
Should be live later this morning, I'll ping back here |
Beta Was this translation helpful? Give feedback.
-
|
Just to follow up, ran into a few issues, aiming for by EOB today. |
Beta Was this translation helpful? Give feedback.
-
|
All files built, just working on getting them up on to Github now.
ETA: ~1.5 hours, if the scripts push through as hoped. I have to go do some childcare but will try to check in a little later this evening to confirm everything went as planned. I have some new badges on the front of the repo that track the github actions building things now, so if any of those turn red you'll know something went amiss (hopefulyl not :) ). |
Beta Was this translation helpful? Give feedback.
-
|
Alright, a full build was pushed yesterday, but I was looking at the files and it looks like some were skipped during upload for an unknown reason. I'm going to debug that one later today and see what happened. But for now going to close this thread out (well, stop replying here :) ), as at least the shapeName issue is fixed. Should figure out this push issue by EOB today. The repo may have hit a size limit, in which case I might move forward with the refresh described over in the other thread. We'll see what's going on. |
Beta Was this translation helpful? Give feedback.
-
|
Just to fully close the loop on this - this is now all fixed and working as expected, as of a few days ago. |
Beta Was this translation helpful? Give feedback.
Just confirming that, yes, shapeName is missing, which is an error. It didn't get flagged because it's technically optional (i.e., we don't have it for every case), but it should still be present in most cases.
Working on a fix now (may be ~later tonight / tomorrow before everything gets pushed after I get this running).