Skip to content

Temporary: fix CI and dependency "compatibility"#218

Merged
firefly-cpp merged 1 commit intofirefly-cpp:mainfrom
KukovecRok:main
Feb 16, 2026
Merged

Temporary: fix CI and dependency "compatibility"#218
firefly-cpp merged 1 commit intofirefly-cpp:mainfrom
KukovecRok:main

Conversation

@KukovecRok
Copy link
Copy Markdown
Contributor

@firefly-cpp This is only a temporary fix!

There are multiple problems here that need to be addressed (see CI log for details). I'll give a brief explanation:

  1. AttributeError: module 'pkgutil' has no attribute 'ImpImporter' is removed in Python >= 3.12
  2. Older versions of setuptools still reference ImpImporter
  3. Older NumPy versions: pip tried to build from source instead of using a wheel, causing error
  4. ModuleNotFoundError: No module named 'pkg_resources'; pkg_resources is deprecated as an API

The real problematic dependency here is:
geotiler 0.15.1, which has not been updated for 2 years. I've "fixed" pipeline, reducing setuptools to <70, which supports "legacy" behavior.
Pipeline is currently succesful on e.g. "platform linux -- Python 3.12.3, pytest-7.4.4, pluggy-1.5.0"

TLDR; we have mixed dependencies, modern (e.g. NumPy > 2.0.0) and legacy (geotiler).

This needs a longterm fix (replacing/forking/contributing to geotiler, updating NumPy API, ...)

@firefly-cpp
Copy link
Copy Markdown
Owner

Thanks! The only logical decision is to abandon the geotiler as soon as possible.

@firefly-cpp firefly-cpp merged commit a253255 into firefly-cpp:main Feb 16, 2026
2 checks passed
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