Skip to content

Conversation

@noellehmcheng
Copy link

No description provided.

@noellehmcheng
Copy link
Author

noellehmcheng commented Oct 8, 2025

  • remove duplicate unit names when building unit parameter that gets pass to sort function

@noellehmcheng
Copy link
Author

noellehmcheng commented Oct 14, 2025

#26
This pr address a circular dependency:

  1. Sorter: produces stratigraphic column, but requires contacts
  2. BasalContacts: produces contacts, but requires stratigraphic column
  3. Contacts: needed by sorter to produce stratigraphic column

#27
This pr separates contact and basal contact extractors, but QGIS crashes when running these algorithms in sequence due to a PROJ error: osgeo::proj::io::DatabaseContext::Private::open
This pr is NOT for merging, just for testing purposes.

#28
This PR implements a custom dockwidget with separate contact and basal contact extractor algorithms that run successfully without crashes.

@noellehmcheng noellehmcheng marked this pull request as ready for review October 16, 2025 02:57
@lachlangrose lachlangrose merged commit cda2efd into noelle/thickness_calculator Oct 16, 2025
6 checks passed
@lachlangrose lachlangrose deleted the fix_processing_tools/sorter branch October 16, 2025 06:11
@noellehmcheng noellehmcheng restored the fix_processing_tools/sorter branch October 16, 2025 06: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.

3 participants