Skip to content

remove or replace cgeo C++ module #27

Description

@abc8747

minisky/tools/src_cpp/setup.py uses distutils.core.Extension, which was deprecated in PEP 632 (Python 3.10). Since we have to build cross-platform wheels soon anyway, we should look into either:

  • use scikit-build-core, specifically pybind11/nanobind (cmake is a headache though)
  • or rewrite it in Rust altogether, and use PyO3/maturin so we can generate .pyi stubs easily. However, compatibility with numpy arrays is unknown (maybe we can borrow some ideas from @xoolive 's thrust)

The latter approach is desired because we the current tangram frontend lacks any syntax highlighting for commands, so maybe we can create a wasm module with Rust. But whether that is really needed is still too early to tell

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions