DHI / DHI/mikecore-python

eum.py: ~120 lines of commented-out, half-translated EUM filter API stubs — port or delete

Abierto
#48 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Python
Estrellas
5
Forks
1
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

`mikecore/eum.py` carries roughly 120 lines of commented-out, half-translated C# declarations — from about line 1904 to about line 2025 — each carrying a `# TODO: Pending` marker. Nine of the module's TODOs live inside them. They look like this:

```python
# #/
# #/ TODO: Pending
# #/
# @staticmethod
# def eumGetFilterCount(); -> int:
#
# private static bool _eumGetFilterSeq(SeqNo: int, out FtKey: int, out IntPtr lpFtDesc);
```

They are not valid Python even uncommented — the signatures are a mix of Python and C# (`def eumGetFilterCount(); -> int:`, `private static bool`, `out IntPtr`), so they are a transcription of the original API surface rather than code that ever ran.

The affected group is the EUM filter/sequence family (`eumGetFilterCount`, `eumGetFilterSeq`, and neighbours).

Worth deciding once, either way:

- **Port them**, if anything needs to enumerate EUM filters from Python; or
- **Delete them**, and note in the module docstring that the filter API is not wrapped.

Either outcome removes nine TODOs and about 120 lines of noise from the largest module in the package. Leaving them is the only option that keeps implying work is planned when none is.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Inspecciona el bloque comentado de filtros/secuencia de EUM en mikecore/eum.py alrededor de las líneas 1904–2025 y la API envuelta circundante para determinar si los llamadores de Python necesitan enumerar los filtros. Se considera completado cuando se sustituyan las declaraciones por wrappers válidos de Python o se elimine el bloque, sus nueve TODOs y se documente la API de filtros no envuelta en el docstring del módulo.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
api, backend
Tipo de issue
Refactorización
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Activo
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.