isce-framework / isce-framework/isce3
Consider adding support for numpy vectorize
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 239
- Forks
- 90
- Avg merge
- 13d 1h
- Merged PRs (30d)
- 5
Description
A number of geometry operations could be vectorized for use with numpy arrays directly following: https://pybind11.readthedocs.io/en/stable/advanced/pycpp/numpy.html#vectorizing-functions
Looping over the arrays in python works for now but performance could be lot better if the loop happened in C++.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the geometry operations that currently require Python loops and the linked pybind11 NumPy vectorization guidance. Done means the relevant operations accept NumPy arrays directly, with iteration performed in C++ for improved performance; no specific files or tests are named.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp, numpy, python
- Domain
- performance
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100