test_eum.py fails on master: eumItem is missing 110321, and CI pins an older native version that hides it
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 45/100
Direzione di ricerca
Inizia da tests/test_eum.py::TestEUM::test_wrapper e mikecore/eum.py, quindi confronta le versioni dei pacchetti nativi fissate in .github/workflows/test_linux.yml e build_linux.yml. Il risultato immediato dovrebbe coprire EUM item 110321 e riprodurre la suite sulla versione nativa distribuita; il lavoro più ampio richiede una sorgente EUM versionata e leggibile dalle macchine, bindings generati e una gestione tollerante degli ids sconosciuti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
tests/test_eum.py::TestEUM::test_wrapper fails on master today:
ValueError: 110321 is not a valid eumItem
It fails on the first assertion, at tests/test_eum.py:8. eumWrapper.CreateItemHashtable() asks the native EUM library for every item it knows and converts each id through eumItem(...). The native library reports item 110321, "Carbon Emmision Factor". The Python eumItem enum in mikecore/eum.py ends at eumIThickness = 110320, so the conversion raises and the whole test dies before reaching anything it meant to check.
Why CI does not see it
The workflows pin different versions of the same native package:
.github/workflows/test_linux.yml:31—nuget install DHI.MikeCore.Linux.rhel7 -Version 20.0.0.github/workflows/build_linux.yml:31—nuget install DHI.MikeCore.Linux.ubuntu -Version 22.1.0
So the tests run against a native library old enough not to know 110321, while the wheel that ships is built against one that does. Green CI, red suite for anyone installing the released package. Aligning the two versions would make CI reproduce this immediately.
The underlying shape of the problem
mikecore/eum.py carries a hand-transcribed copy of the EUM item and unit tables — 607 eumItem members — with the maintenance burden stated in the source itself, twice (lines 9 and 611):
# Must be updated with every new release, or if the EUM.xml is updated
Every binding to this library repeats that transcription and then drifts on its own schedule, so this failure is not a one-off missing constant — it is the mode this design fails in. Any new item added on the native side breaks CreateItemHashtable() in any binding that has not been hand-updated, and the break is total rather than per-item, because one unmappable id takes down the whole table.
Two fixes, and they are independent
Immediate: add the missing member so the current native library round-trips:
eumICarbonEmmisionFactor = 110321
(spelling copied from the native key, which has the typo)
Structural: stop the next one from being a test failure at all.
The root need is a canonical, persistent, machine-readable source for EUM codes — a stable public URL listing every item and unit id with its key — that bindings can generate from and that users can look a code up in. Today there is no such page: MIKE Core SDK / EUM describes what EUM is, and mikeio's EUM guide shows how to search types from Python, but neither enumerates the codes. The only complete list is the one each binding transcribes by hand, which is why they drift.
The model to copy is the CF standard name table: a numbered version, a persistent URL per version, a machine-readable XML artifact alongside the human-readable page, and a documented process for adding a name. Every client generates from it instead of transcribing it, and a citation like "CF standard names v88" is unambiguous years later. EUM has the same shape of problem — a controlled vocabulary of quantities and units consumed by many independent clients — and none of that infrastructure.
With that in place, eum.py becomes generated from a versioned artifact rather than maintained by hand, and a native version bump regenerates instead of breaking.
Until then, a smaller mitigation is worth having regardless: make id-to-enum conversion tolerant, so an id the enum does not know yields an unknown/passthrough item instead of raising and taking the whole hashtable down with it. One unrecognised code should not cost every other code in the table.
The immediate fix unblocks the suite. The canonical source is what stops this recurring on every release.
- Lingua principale
- Python
- Stelle
- 5
- Fork
- 1
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di DHI/mikecore-python
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
DHI/mikecore-python#49 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 42/100
DHI/mikecore-python#54 · 2 commenti ·
-
Remaining TODOs in production code: platform notes, licensing questions, and one likely-stale marker Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100
DHI/mikecore-python#51 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
DHI/mikecore-python#50 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 45/100
DHI/mikecore-python#48 ·
Tutte le issue di DHI/mikecore-python
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
xinnan-tech/xiaozhi-fde-talk#263 ·
-
rules
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
huggingface/Repo2RLEnv#163 · 1 commento ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 95/100
huggingface/sentence-transformers#4074 ·
-
comp/dashboard invalid P3
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
NousResearch/hermes-agent#121143 ·