wof emit fails to handle alt files (SPR) #37

Activity

1. thisisaaronland

   self-assigned this

   on Sep 18, 2025
2. thisisaaronland

   added

   bugSomething isn't workingSomething isn't working

   on Sep 18, 2025
3. thisisaaronland commented on Sep 18, 2025

   

   thisisaaronland

   on Sep 18, 2025

   MemberAuthor

   More actions

   This looks like a go-writer-geoparquet issue:

   > git grep 'Failed to derive SPR from'
   vendor/github.com/whosonfirst/go-writer-geoparquet/v3/geoparquet.go:                    return 0, fmt.Errorf("Failed to derive SPR from %s, %w", key, err)
   
4. thisisaaronland commented on Sep 18, 2025

   

   thisisaaronland

   on Sep 18, 2025

   MemberAuthor

   More actions

   Actually, the problem seems to be in tidwall/sjson assigning the (WOF) SPR as initial properties. For example, in go-writer-geoparquet/geoparquet.go:

      186 |        wtf_body, err := json.Marshal(wof_spr)
      187 |
      188 |	if err != nil {
      189 |                return 0, err
      190 |	}
      191 |
      192 |        if strings.Contains(string(body), "WOFParentId"){
      193 |                slog.Info("WTF 0", "body", string(wtf_body))
      194 |	}
      195 |
      196 |	// START OF wrangle properties in to something GeoParquet can work with
      197 |
      198 |	old_props := gjson.GetBytes(body, "properties")
      199 |
      200 |	body, err = sjson.SetBytes(body, "properties", wof_spr)
      201 |
      202 |	if err != nil {
      203 |                return 0, fmt.Errorf("Failed to update properties for %s, %w", key, err)
      204 |        }
      205 |
      206 |	if strings.Contains(string(body), "WOFParentId"){
      207 |                slog.Info("WTF 1", "body", string(body))
      208 |        }
   

   WTF 0 is not triggered but WTF 1 is always triggered.
5. straup

   added

   emit

   on Oct 8, 2025