Skip to content

fully onboard required code from unmaintained openeo-processes-python - #544

Merged
soxofaan merged 6 commits into
masterfrom
processes_import
Oct 16, 2026
Merged

soxofaan merged 6 commits into
masterfrom
processes_import

Conversation

@jdries

@jdries jdries commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

One possible solution for: #152
And unblocks python versions >3.11

This is what it would look like if we import all whitelisted processes and unit tests.
Code volume could perhaps be less if we deviate from a strict copy.

@jdries
jdries requested review from JeroenVerstraelen and soxofaan and removed request for soxofaan October 13, 2026 18:28

@soxofaan soxofaan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some notes

Comment thread tests/test_vectorcube_from_fiona.py Outdated
Comment thread openeo_driver/processgraph/registry.py
Comment thread openeo_driver/processgraph/registry.py
Comment thread openeo_driver/vendor/processes_python/__init__.py
@soxofaan

Copy link
Copy Markdown
Member

And unblocks python versions >3.11

what is the blocker or failure here?

@jdries

jdries commented Oct 14, 2026

Copy link
Copy Markdown
Contributor Author

And unblocks python versions >3.11

what is the blocker or failure here?

removal of pkg_resources (which is like used in a single line somewhere)

@jdries
jdries marked this pull request as ready for review October 15, 2026 08:29
@jdries

jdries commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

ready to merge, this should now be a 'no change' kind of PR, only moving existing unmaintained code under our maintenance scope

@soxofaan
soxofaan merged commit 62f30c8 into master Oct 16, 2026
3 of 6 checks passed
@soxofaan
soxofaan deleted the processes_import branch October 16, 2026 07:31
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.

2 participants