Skip to content
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

[5pt] ValueError: Object not recognized source of Features (Head bridges) #1457

Open
RobHanna-NOAA opened this issue Mar 10, 2025 · 3 comments · May be fixed by #1463
Open

[5pt] ValueError: Object not recognized source of Features (Head bridges) #1457

RobHanna-NOAA opened this issue Mar 10, 2025 · 3 comments · May be fixed by #1463
Assignees
Labels
bug Something isn't working

Comments

@RobHanna-NOAA
Copy link
Contributor

In the latest hand_4_6_0_0, over 350 errors saying ValueError: Object is not a recognized source of Features can be found in the all_errors_from_logs.log file.

@RobHanna-NOAA RobHanna-NOAA added the bug Something isn't working label Mar 10, 2025
@mluck
Copy link
Contributor

mluck commented Mar 10, 2025

Here's an example from the log file:

Burn in bridges 17100305 0
Traceback (most recent call last):
  File "/foss_fim/src/heal_bridges_osm.py", line 263, in <module>
    process_bridges_in_huc(**args)
  File "/foss_fim/src/heal_bridges_osm.py", line 132, in process_bridges_in_huc
    lidar_osm_gdf, updated_hand_grid_array = process_lidar_osm(
  File "/foss_fim/src/heal_bridges_osm.py", line 89, in process_lidar_osm
    stats = zonal_stats(
  File "/usr/local/lib/python3.10/dist-packages/rasterstats/main.py", line 52, in zonal_stats
    return list(gen_zonal_stats(*args, **kwargs))
  File "/usr/local/lib/python3.10/dist-packages/rasterstats/main.py", line 178, in gen_zonal_stats
    features_iter = read_features(vectors, layer)
  File "/usr/local/lib/python3.10/dist-packages/rasterstats/io.py", line 136, in read_features
    raise ValueError("Object is not a recognized source of Features")
ValueError: Object is not a recognized source of Features

@RobHanna-NOAA
Copy link
Contributor Author

If this is considered to be an acceptable error, which we have a good handful of, we have some options.

  1. if possible, maybe we can add it to our fim exit status code system so we can easily identify those if we even want to.
  2. We don't' even need to record it in logs anywhere if you like.
  3. We can change the output message to not have the word "error" in it but change it to the word "warning". Our post processing system looks through all logs looking for the word "error" and "warning"

@RobHanna-NOAA RobHanna-NOAA changed the title [5pt] ValueError: Object not recognized source of Features [5pt] ValueError: Object not recognized source of Features (Head bridges) Mar 11, 2025
@RobHanna-NOAA
Copy link
Contributor Author

Yes. it is an actual error and over 350 HUCs failed at the unit_errors in the hand_4_6_0_0 build. I missed this detail earlier (gulp).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants