Skip to content

fix: OSD data aggregation and CEC duplicate column issues#360

Open
jjmaynard wants to merge 2 commits intomainfrom
fix/OSD-infilling-error
Open

fix: OSD data aggregation and CEC duplicate column issues#360
jjmaynard wants to merge 2 commits intomainfrom
fix/OSD-infilling-error

Conversation

@jjmaynard
Copy link
Collaborator

  • Fixed array length inconsistencies in OSD infilling by using stored horizon depths from hzb_lyrs instead of muhorzdata_pd_group
  • Applied fix to sand/clay/texture aggregation (lines 919-942)
  • Applied fix to LAB/Munsell aggregation (lines 775-806)
  • Fixed CEC duplicate column issue in process_horizon_data by filling cec7_r in-place before rename
  • Fixed CEC/pH/EC empty value logic to correctly handle dictionary structures
  • CEC values now return as numeric instead of strings"

- Fixed CEC duplicate column issue in process_horizon_data by filling cec7_r in-place before rename
- CEC values now return as numeric instead of strings"
- Fixed array length inconsistencies in OSD infilling by using stored horizon depths from hzb_lyrs instead of muhorzdata_pd_group
- Applied fix to sand/clay/texture aggregation (lines 919-942)
- Applied fix to LAB/Munsell aggregation (lines 775-806)
@garobrik garobrik force-pushed the fix/OSD-infilling-error branch from bfe86d4 to b2ff977 Compare February 27, 2026 00:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant