cellpycore 0.2.1__tar.gz → 0.2.3__tar.gz
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- cellpycore-0.2.3/.gitattributes +14 -0
- cellpycore-0.2.3/.issueflows/03-solved-issues/issue136_original.md +60 -0
- cellpycore-0.2.3/.issueflows/03-solved-issues/issue136_plan.md +87 -0
- cellpycore-0.2.3/.issueflows/03-solved-issues/issue136_status.md +18 -0
- cellpycore-0.2.3/.issueflows/03-solved-issues/issue139_original.md +61 -0
- cellpycore-0.2.3/.issueflows/03-solved-issues/issue139_plan.md +119 -0
- cellpycore-0.2.3/.issueflows/03-solved-issues/issue139_status.md +18 -0
- cellpycore-0.2.3/.issueflows/04-designs-and-guides/legacy-bridge-test-id.md +15 -0
- cellpycore-0.2.3/.issueflows/04-designs-and-guides/legacy-raw-energy-mapping.md +24 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/this-project.md +10 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/HISTORY.md +16 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/PKG-INFO +1 -1
- {cellpycore-0.2.1 → cellpycore-0.2.3}/graphify-out/.graphify_labels.json +28 -2
- {cellpycore-0.2.1 → cellpycore-0.2.3}/graphify-out/GRAPH_REPORT.md +277 -176
- cellpycore-0.2.3/graphify-out/graph.html +305 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/graphify-out/graph.json +10799 -5012
- {cellpycore-0.2.1 → cellpycore-0.2.3}/graphify-out/manifest.json +66 -38
- {cellpycore-0.2.1 → cellpycore-0.2.3}/pyproject.toml +1 -1
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/cell_core.py +23 -3
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/legacy/headers.py +2 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/legacy/mapping.py +14 -14
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/merge.py +28 -1
- {cellpycore-0.2.1 → cellpycore-0.2.3}/uv.lock +1 -1
- cellpycore-0.2.1/graphify-out/graph.html +0 -305
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.aliases +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/rules/cellpy-core-migration.mdc +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/rules/graphify.mdc +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/rules/issueflow-rules.mdc +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/rules/kiss.mdc +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/rules/this-project.mdc +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/caveman/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/grill-me/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-archive/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-cleanup/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-close/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-comments/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-cycle/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-epic/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-fix/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-graphify/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-history-update/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-init/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-pause/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-pick/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-plan/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-start/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-status/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-version-bump/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.cursor/skills/iflow-yolo/SKILL.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.gitignore +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/00-tools/.gitkeep +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/00-tools/README.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/01-current-issues/.gitkeep +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/02-partly-solved-issues/.gitkeep +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/02-partly-solved-issues/issue100_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/02-partly-solved-issues/issue100_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/02-partly-solved-issues/issue100_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/.gitkeep +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue10_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue10_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue10_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue110_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue110_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue110_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue112_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue112_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue112_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue114_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue114_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue114_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue115_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue115_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue115_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue116_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue116_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue116_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue117_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue117_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue117_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue118_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue118_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue118_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue121_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue121_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue121_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue12_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue12_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue12_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue13_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue13_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue13_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue21_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue21_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue21_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue22_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue22_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue22_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue23_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue23_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue23_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue24_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue24_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue29_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue29_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue29_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue30_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue30_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue30_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue32_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue32_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue32_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue34_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue34_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue36_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue36_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue39_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue39_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue39_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue40_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue40_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue40_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue41_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue41_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue41_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue42_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue42_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue42_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue43_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue43_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue43_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue44_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue44_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue45_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue45_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue45_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue50_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue50_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue50_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue54_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue54_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue54_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue55_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue55_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue55_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue56_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue56_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue56_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue62_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue62_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue62_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue64_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue64_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue64_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue66_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue66_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue66_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue67_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue67_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue67_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue68_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue68_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue68_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue69_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue69_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue69_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue70_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue70_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue70_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue72_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue72_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue77_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue77_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue77_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue82_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue82_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue84_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue84_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue84_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue86_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue86_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue86_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue91_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue91_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue91_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue95_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue95_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue95_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue96_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue96_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue96_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue97_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue97_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue97_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue98_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue98_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue98_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue99_original.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue99_plan.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/03-solved-issues/issue99_status.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/.gitkeep +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/bdf-io-placement.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/cellpy-core-integration-into-cellpy.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/cellpy-core-integration-roadmap.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/cellpy-core-migration.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/code-review-2026-07.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/column-headers-review.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/metadata-scaffolding.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/nom-cap-naming.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/rawcols-dtype-map.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/release-procedure.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/selector-dead-code-deferral.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/step-c-rate-split.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/step-table-polars-migration.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/summary-extractors.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/test-data-and-fixtures.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/test-metadata-and-merging.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/04-designs-and-guides/zensical-docs.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/05-epics/.gitkeep +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.issueflows/config.toml +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.pre-commit-config.yaml +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.python-version +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.readthedocs.yaml +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/.vscode/settings.json +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/AGENTS.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/LICENSE +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/README.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/ROADMAP.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/SCRATCHPAD.md +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/graphify-out/.graphify_root +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/__init__.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/config.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/curves.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/exceptions.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/extractors.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/legacy/__init__.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/legacy/limits.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/legacy/meta.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/legacy/meta_mapping.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/legacy/mock_core.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/legacy/selectors.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/metadata/__init__.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/metadata/io.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/metadata/models.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/py.typed +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/settings_base.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/summarizers.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/testing/__init__.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/testing/mock_data.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/timestamps.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/units/__init__.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/units/converters.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/src/cellpycore/units/spec.py +0 -0
- {cellpycore-0.2.1 → cellpycore-0.2.3}/zensical.toml +0 -0
|
@@ -0,0 +1,14 @@
|
|
|
1
|
+
# GitHub Linguist: keep language stats focused on library source.
|
|
2
|
+
# Without this, graphify-out/graph.html (~3.8MB) dominates as HTML.
|
|
3
|
+
|
|
4
|
+
graphify-out/** linguist-generated
|
|
5
|
+
docs/** linguist-documentation
|
|
6
|
+
tests/** linguist-documentation
|
|
7
|
+
.issueflows/** linguist-documentation
|
|
8
|
+
dev/** linguist-documentation
|
|
9
|
+
scripts/** linguist-documentation
|
|
10
|
+
|
|
11
|
+
# Cross-platform line endings for shell helpers
|
|
12
|
+
.aliases text eol=lf
|
|
13
|
+
*.sh text eol=lf
|
|
14
|
+
*.lock text eol=lf
|
|
@@ -0,0 +1,60 @@
|
|
|
1
|
+
# Issue #136: Legacy bridge: preserve test_id through steps/summary output; legacy-schema support for merge_data
|
|
2
|
+
|
|
3
|
+
Source: https://github.com/cellpy/cellpy-core/issues/136
|
|
4
|
+
|
|
5
|
+
## Original issue text
|
|
6
|
+
|
|
7
|
+
## Summary
|
|
8
|
+
|
|
9
|
+
The `OldCellpyCellCore` legacy bridge strips `test_id` from the outbound
|
|
10
|
+
step/summary frames, which degrades multi-test (campaign-merged) objects to
|
|
11
|
+
cycle-only summary grouping. cellpy #507 works around it; these requests make
|
|
12
|
+
the workaround unnecessary and unblock campaign merges that keep original
|
|
13
|
+
per-test cycle numbers.
|
|
14
|
+
|
|
15
|
+
## Details
|
|
16
|
+
|
|
17
|
+
1. **Outbound `test_id` stripping.** `_native_steps_to_legacy` re-projects to a
|
|
18
|
+
fixed legacy column order (cell_core.py ~781-815) and the summary path
|
|
19
|
+
filters to `_legacy_summary_column_order` (~928-1065); `test_id` is in
|
|
20
|
+
`NATIVE_ONLY_STEP` / `NATIVE_ONLY_CYCLE` (mapping.py ~191-196, 227-231). The
|
|
21
|
+
inbound side is fine: a raw `test_id` column passes through (exception
|
|
22
|
+
lists, mapping.py ~129-144) and `make_step_table` groups on
|
|
23
|
+
`(test_id, cycle_num, step_num)` (proven by the cycler_small golden).
|
|
24
|
+
2. **Bridge summary rebuilds native steps from the stripped legacy steps**
|
|
25
|
+
(`make_core_summary`, cell_core.py ~1010-1012), so
|
|
26
|
+
`use_tid` (summarizers.py ~792) is always False through the bridge —
|
|
27
|
+
cycle-only grouping, no per-test windowing, and cross-test collision when
|
|
28
|
+
cycle numbers overlap. cellpy #507's workaround: re-stamp `test_id` onto the
|
|
29
|
+
legacy steps frame post-bridge (depends on the inbound step rename staying
|
|
30
|
+
rename-only with no column filtering — please treat that as a contract or
|
|
31
|
+
provide the fix below).
|
|
32
|
+
3. **`_add_legacy_summary_cruft`** (cell_core.py ~960-978) does global pandas
|
|
33
|
+
cumsums (cumulated CE, shifted_*, cumulated_ric*) even when the engine
|
|
34
|
+
windowed per test — these columns are wrong for multi-test objects.
|
|
35
|
+
4. **`merge_data(schema=<legacy Schema>)` raises `AttributeError`** —
|
|
36
|
+
`HeadersNormal` lacks the native attribute names (`datapoint_num`,
|
|
37
|
+
`cycle_num`, `test_id`), and passing the native schema instead silently
|
|
38
|
+
skips offsets on legacy-named frames (`_max_column` returns 0 for missing
|
|
39
|
+
columns). Either document legacy schemas as unsupported for merge/update, or
|
|
40
|
+
add attribute aliases so bridge consumers can call
|
|
41
|
+
`merge_data`/`update_data` directly (cellpy #507 mirrors the semantics in
|
|
42
|
+
pandas instead).
|
|
43
|
+
|
|
44
|
+
## Requests
|
|
45
|
+
|
|
46
|
+
- [ ] Carry `test_id` through the legacy step output (or have
|
|
47
|
+
`make_core_summary` reuse the native steps it already computed instead of
|
|
48
|
+
rebuilding from the stripped legacy frame).
|
|
49
|
+
- [ ] Optionally expose `test_id` in the legacy summary output behind a flag.
|
|
50
|
+
- [ ] Window `_add_legacy_summary_cruft` per `test_id` when present.
|
|
51
|
+
- [ ] Resolve the `merge_data` legacy-schema story (docs or aliases).
|
|
52
|
+
- [ ] Together these unblock `renumber_cycles=False` (original per-test cycle
|
|
53
|
+
numbers) for cellpy's campaign merge ahead of the native-schema path
|
|
54
|
+
(cellpy #511).
|
|
55
|
+
|
|
56
|
+
## Refs
|
|
57
|
+
|
|
58
|
+
cellpy jepegit/cellpy#507 (campaign merge + workaround), jepegit/cellpy#511
|
|
59
|
+
(native schema opt-in), core tests test_golden.py (cycler_small composite-key
|
|
60
|
+
golden), test_merge.py / test_e2e.py (native multi-test invariants).
|
|
@@ -0,0 +1,87 @@
|
|
|
1
|
+
# Issue #136 — plan
|
|
2
|
+
|
|
3
|
+
Source: https://github.com/cellpy/cellpy-core/issues/136
|
|
4
|
+
|
|
5
|
+
## Goal
|
|
6
|
+
|
|
7
|
+
Legacy bridge keeps `test_id` on outbound steps (and summary when present) so
|
|
8
|
+
campaign-merged multi-test objects get per-test summary windowing through the
|
|
9
|
+
bridge — no cellpy #507 re-stamp workaround. Also fix legacy-cruft cumsums and
|
|
10
|
+
clarify `merge_data` + legacy schema.
|
|
11
|
+
|
|
12
|
+
## Constraints
|
|
13
|
+
|
|
14
|
+
- Back-compat: single-test objects already have `test_id=0` on raw; emitting
|
|
15
|
+
`test_id` on steps/summary is **additive** (extra column). Goldens that pin
|
|
16
|
+
exact column sets may need updates.
|
|
17
|
+
- Mapping totality (`NATIVE_ONLY_*` / attr maps) must stay consistent.
|
|
18
|
+
- Do not change native engine behaviour; bridge-only (+ headers/mapping).
|
|
19
|
+
- Unblocks cellpy #510 V2-15 pin after a **new** cellpycore PyPI release.
|
|
20
|
+
|
|
21
|
+
### Prior art
|
|
22
|
+
|
|
23
|
+
| Hit | Role | Plan |
|
|
24
|
+
|-----|------|------|
|
|
25
|
+
| [`OldCellpyCellCore._native_steps_to_legacy`](../../src/cellpycore/cell_core.py) / `_legacy_step_column_order` | Drops unknown cols via fixed order | **Extend** order with `test_id` when present |
|
|
26
|
+
| [`make_core_summary`](../../src/cellpycore/cell_core.py) rebuilds native steps from legacy `data.steps` | `use_tid` false after strip | Fixed once steps carry `test_id` through rename |
|
|
27
|
+
| [`_add_legacy_summary_cruft`](../../src/cellpycore/cell_core.py) | Global pandas cumsums | **Window** by `test_id` when column present |
|
|
28
|
+
| [`NATIVE_ONLY_STEP` / `NATIVE_ONLY_CYCLE`](../../src/cellpycore/legacy/mapping.py) | Declares `test_id` intentionally unbridged | **Remove** `test_id`; add identity scalar/cycle pairs |
|
|
29
|
+
| [`HeadersStepTable` / `HeadersSummary`](../../src/cellpycore/legacy/headers.py) | No `test_id` field today | **Add** `test_id: str = "test_id"` |
|
|
30
|
+
| cellpy [`cellreader.make_step_table` re-stamp](../../../cellpy/cellpy/readers/cellreader.py) (~2293) | Workaround | Becomes redundant after this; leave for a cellpy follow-up |
|
|
31
|
+
| [`merge_data`](../../src/cellpycore/merge.py) | Expects native `Schema` attrs | **Document + hard error** on legacy header schema (cellpy already uses pandas merge) |
|
|
32
|
+
| Toolbox `00-tools/` | Empty | None |
|
|
33
|
+
| Graph | Bridge / mapping communities | Touch `cell_core` + `legacy/mapping` + `legacy/headers` |
|
|
34
|
+
|
|
35
|
+
## Approach
|
|
36
|
+
|
|
37
|
+
1. **Steps outbound:** Add `HeadersStepTable.test_id`. Put `test_id` early in
|
|
38
|
+
`_legacy_step_column_order` (before cycle/step). Add identity
|
|
39
|
+
`STEP_SCALAR_PAIRS` entry `("test_id", "test_id")`; drop `test_id` from
|
|
40
|
+
`NATIVE_ONLY_STEP`. Update attr maps (`LEGACY_ATTR_TO_SCHEMA` /
|
|
41
|
+
`LEGACY_ATTR_UNMAPPED`) for the new header field.
|
|
42
|
+
2. **Summary outbound:** Add `HeadersSummary.test_id`. Include in
|
|
43
|
+
`_legacy_summary_column_order` (near `cycle_index`) when present after rename.
|
|
44
|
+
Add identity `CYCLE_PAIRS` entry; drop `test_id` from `NATIVE_ONLY_CYCLE`.
|
|
45
|
+
No flag — always pass through when the native summary has it (single-test →
|
|
46
|
+
all `0`).
|
|
47
|
+
3. **Cruft:** In `_add_legacy_summary_cruft`, if `test_id` in summary columns and
|
|
48
|
+
`nunique > 1` (or always when column present): compute cumsums / shifted /
|
|
49
|
+
ric **per group**; else keep today's global behaviour.
|
|
50
|
+
4. **`merge_data` + legacy schema:** At entry, if `schema.raw` is
|
|
51
|
+
`HeadersNormal` (or lacks `datapoint_num` / `cycle_num` / `test_id`), raise
|
|
52
|
+
`TypeError` with a clear message: use `config.default_schema()` / native
|
|
53
|
+
`Schema`. Document in `merge_data` docstring. No full legacy-schema merge
|
|
54
|
+
implementation in this issue (cellpy does not call it).
|
|
55
|
+
5. **Tests:** Bridge multi-test fixture (or reuse schema/e2e multi-test raw) →
|
|
56
|
+
`make_core_step_table` / `make_core_summary` → assert `test_id` on steps and
|
|
57
|
+
summary `{0,1}`; assert summary cumulatives restart per test. Mapping
|
|
58
|
+
totality tests update with the exception-set moves. One test that
|
|
59
|
+
`merge_data(..., schema=legacy.schema)` raises the new TypeError.
|
|
60
|
+
|
|
61
|
+
## Files to touch
|
|
62
|
+
|
|
63
|
+
| Path | Change |
|
|
64
|
+
|------|--------|
|
|
65
|
+
| `src/cellpycore/legacy/headers.py` | `test_id` on step + summary headers |
|
|
66
|
+
| `src/cellpycore/legacy/mapping.py` | pairs / `NATIVE_ONLY_*` / attr maps + comments |
|
|
67
|
+
| `src/cellpycore/cell_core.py` | column order, cruft windowing |
|
|
68
|
+
| `tests/test_*.py` (bridge / mapping / merge) | new + update totality |
|
|
69
|
+
| `CHANGELOG` / HISTORY if present | note for 0.2.2 release |
|
|
70
|
+
|
|
71
|
+
## Test strategy
|
|
72
|
+
|
|
73
|
+
```bash
|
|
74
|
+
uv run pytest
|
|
75
|
+
uv run ruff check && uv run ruff format --check
|
|
76
|
+
```
|
|
77
|
+
|
|
78
|
+
Focus: new bridge multi-test test + `tests/test_header_mapping.py` totality +
|
|
79
|
+
`tests/test_merge.py` legacy-schema error.
|
|
80
|
+
|
|
81
|
+
## Decisions (confirmed 2026-07-17)
|
|
82
|
+
|
|
83
|
+
1. **Summary `test_id`:** always when present (no flag).
|
|
84
|
+
2. **`merge_data` legacy schema:** hard `TypeError` + docstring (also
|
|
85
|
+
`update_data`).
|
|
86
|
+
3. **Cruft windowing:** when `test_id` column exists (groupby).
|
|
87
|
+
4. **Goldens:** regenerate stage-B step/summary snapshots (additive `test_id`).
|
|
@@ -0,0 +1,18 @@
|
|
|
1
|
+
# Issue #136 — status
|
|
2
|
+
|
|
3
|
+
- [x] Done
|
|
4
|
+
|
|
5
|
+
## What's done
|
|
6
|
+
|
|
7
|
+
- Plan confirmed (2026-07-17): carry `test_id` on steps+summary; window cruft;
|
|
8
|
+
hard-error legacy schema on merge/update; regenerate goldens.
|
|
9
|
+
- Implemented on `136-legacy-bridge-preserve-test-id`.
|
|
10
|
+
- Version bumped `0.2.1` → `0.2.2` (static `pyproject.toml`); HISTORY promoted.
|
|
11
|
+
- `uv run pytest` 261 passed; ruff clean.
|
|
12
|
+
- Close: commit + PR; PyPI release via `release` from `main` after merge
|
|
13
|
+
(see `release-procedure.md`).
|
|
14
|
+
|
|
15
|
+
## Remaining work
|
|
16
|
+
|
|
17
|
+
- None for the code issue. Post-merge: `iflow cleanup` + cut GitHub/PyPI
|
|
18
|
+
release for `0.2.2`, then re-pin cellpy.
|
|
@@ -0,0 +1,61 @@
|
|
|
1
|
+
# Issue #139: legacy mapping: no entries for the energy columns (charge_energy_txt -> cumulative_charge_energy)
|
|
2
|
+
|
|
3
|
+
Source: https://github.com/cellpy/cellpy-core/issues/139
|
|
4
|
+
|
|
5
|
+
## Original issue text
|
|
6
|
+
|
|
7
|
+
## Summary
|
|
8
|
+
|
|
9
|
+
`cellpycore.legacy.mapping.LEGACY_ATTR_TO_SCHEMA["raw"]` has no entries for the
|
|
10
|
+
energy columns, although both sides exist:
|
|
11
|
+
|
|
12
|
+
- native schema defines `cumulative_charge_energy` and
|
|
13
|
+
`cumulative_discharge_energy`;
|
|
14
|
+
- cellpy's loader configurations declare `charge_energy_txt` and
|
|
15
|
+
`discharge_energy_txt`, and vendor files carry the data (Maccor `Watt-hr`,
|
|
16
|
+
Neware energy columns, …).
|
|
17
|
+
|
|
18
|
+
```python
|
|
19
|
+
>>> from cellpycore.legacy import mapping
|
|
20
|
+
>>> {k: v for k, v in mapping.LEGACY_ATTR_TO_SCHEMA["raw"].items() if "energy" in k}
|
|
21
|
+
{}
|
|
22
|
+
```
|
|
23
|
+
|
|
24
|
+
## Why it matters now
|
|
25
|
+
|
|
26
|
+
Found while porting loaders to the `harmonize()` framework
|
|
27
|
+
(jepegit/cellpy#559 → #560). Consequences today:
|
|
28
|
+
|
|
29
|
+
1. On the native runtime, `to_native()` passes the unmapped columns through
|
|
30
|
+
under their **legacy** names, so a native frame carries `charge_energy`
|
|
31
|
+
while the schema says the column should be `cumulative_charge_energy`.
|
|
32
|
+
Asking `c.schema.raw` for energy finds nothing.
|
|
33
|
+
2. `harmonize()` drops undeclared columns by design, so a ported loader would
|
|
34
|
+
silently lose energy data unless it declares a passthrough — which is the
|
|
35
|
+
wrong long-term answer for a column the schema already has a name for.
|
|
36
|
+
|
|
37
|
+
## Ask
|
|
38
|
+
|
|
39
|
+
Add the mapping entries:
|
|
40
|
+
|
|
41
|
+
| legacy attribute | native column |
|
|
42
|
+
|---|---|
|
|
43
|
+
| `charge_energy_txt` | `cumulative_charge_energy` |
|
|
44
|
+
| `discharge_energy_txt` | `cumulative_discharge_energy` |
|
|
45
|
+
|
|
46
|
+
Please confirm the reset-granularity convention for the energy columns matches
|
|
47
|
+
the capacity one (cycle-cumulative per direction, per the harmonized-raw spec's
|
|
48
|
+
*Capacity convention*) — the cellpy side will normalise them the same way.
|
|
49
|
+
|
|
50
|
+
## Also unmapped (for a decision, not necessarily this issue)
|
|
51
|
+
|
|
52
|
+
These native raw columns have no legacy attribute mapped to them. Several are
|
|
53
|
+
correctly framework-owned or native-only, but `datetime_txt → epoch_time_utc`
|
|
54
|
+
looks like a genuine omission of the same kind:
|
|
55
|
+
|
|
56
|
+
`epoch_time_utc`, `step_type`, `step_mode`, `step_type_detail`, `cycle_type`,
|
|
57
|
+
`ref_potential`, `step_charge_power`, `step_discharge_power`, `aux_*`,
|
|
58
|
+
`source_*`, `mask`, `test_id`.
|
|
59
|
+
|
|
60
|
+
Cross-repo order per the cellpy 2 release plan §3: core PR → core release →
|
|
61
|
+
cellpy re-pin → cellpy consumes.
|
|
@@ -0,0 +1,119 @@
|
|
|
1
|
+
# Plan — issue #139 (legacy raw energy mapping)
|
|
2
|
+
|
|
3
|
+
## Goal
|
|
4
|
+
|
|
5
|
+
Bridge legacy raw energy columns through the authoritative header map so
|
|
6
|
+
`charge_energy` / `discharge_energy` rename to
|
|
7
|
+
`cumulative_charge_energy` / `cumulative_discharge_energy` (same pattern as
|
|
8
|
+
capacity), enabling `to_native()` / `harmonize()` consumers to find them under
|
|
9
|
+
the native schema names.
|
|
10
|
+
|
|
11
|
+
## Constraints
|
|
12
|
+
|
|
13
|
+
- Keep change to `mapping.py` + its existing totality / spot-check tests.
|
|
14
|
+
No loader work, no engine changes, no cycle/step energy columns.
|
|
15
|
+
- Totality discipline (#116): every legacy attr / column value is either mapped
|
|
16
|
+
or in an exception set — never both, never neither.
|
|
17
|
+
- Value-level pairs (`RAW_PAIRS`) drive DataFrame renames; attribute table
|
|
18
|
+
(`LEGACY_ATTR_TO_SCHEMA`) must stay in sync (`test_legacy_attr_matches_value_mapping`).
|
|
19
|
+
- Out of scope (issue itself): `datetime_txt → epoch_time_utc` and the other
|
|
20
|
+
unmapped native-only list — track separately if wanted.
|
|
21
|
+
- Cross-repo: core PR → core release → cellpy re-pin → cellpy consumes
|
|
22
|
+
(cellpy #560). This issue is the core PR only.
|
|
23
|
+
- Spec already states the reset convention (see Approach); record that as the
|
|
24
|
+
answer to the issue's ask — do not invent a new convention here.
|
|
25
|
+
|
|
26
|
+
### Prior art
|
|
27
|
+
|
|
28
|
+
- `RAW_PAIRS` / `LEGACY_ATTR_TO_SCHEMA["raw"]` / exception sets —
|
|
29
|
+
[`src/cellpycore/legacy/mapping.py`](src/cellpycore/legacy/mapping.py).
|
|
30
|
+
Mirror capacity entries (`charge_capacity_txt` →
|
|
31
|
+
`cumulative_charge_capacity`).
|
|
32
|
+
- Totality + spot-check tests —
|
|
33
|
+
[`tests/test_header_mapping.py`](tests/test_header_mapping.py)
|
|
34
|
+
(`test_raw_*_totality`, `test_known_translations`,
|
|
35
|
+
`test_legacy_attr_*`).
|
|
36
|
+
- Spec Capacity convention —
|
|
37
|
+
[`docs/specifications/harmonized-raw.md`](docs/specifications/harmonized-raw.md)
|
|
38
|
+
(§ Capacity convention): energy columns share cycle-cumulative / per-direction
|
|
39
|
+
semantics with capacity.
|
|
40
|
+
- Issue #42 (solved): engine reset-granularity normalizer already covers
|
|
41
|
+
`cumulative_*_energy` when source granularity differs.
|
|
42
|
+
- Fixture helper already assumes the rename —
|
|
43
|
+
[`dev/make_harmonized_raw.py`](dev/make_harmonized_raw.py)
|
|
44
|
+
(`charge_energy` → `cumulative_charge_energy`).
|
|
45
|
+
- Issue #116 (solved): attribute-table + `LEGACY_ATTR_UNMAPPED` discipline.
|
|
46
|
+
- Toolbox: none relevant.
|
|
47
|
+
- Graph: community 87 (`mapping.py`) / 82 (legacy headers) — no extra modules.
|
|
48
|
+
|
|
49
|
+
**Note vs architecture-plan D3:** `cellpy2-native-headers-migration-plan.md`
|
|
50
|
+
still lists raw energy as passthrough ("not synthesized — different reset
|
|
51
|
+
semantics"). That is **stale relative to the harmonized-raw Capacity
|
|
52
|
+
convention + #42**. Mapping is a rename of already-supplied columns, not
|
|
53
|
+
synthesis. After Accept, add a one-line correction in
|
|
54
|
+
`.issueflows/04-designs-and-guides/` (or a comment in the plan status) so
|
|
55
|
+
future agents do not re-block the bridge; updating the architecture-plan
|
|
56
|
+
doc can be a tiny follow-up outside this PR if desired.
|
|
57
|
+
|
|
58
|
+
## Approach
|
|
59
|
+
|
|
60
|
+
1. **Confirm convention (recorded decision):** energy matches capacity —
|
|
61
|
+
cumulative per cycle, per direction, reset at cycle boundary
|
|
62
|
+
(`harmonized-raw.md` Capacity convention). Cellpy loaders normalize the
|
|
63
|
+
same way; if a source is step-cumulative, #42's normalizer is the path,
|
|
64
|
+
not leaving the columns unmapped.
|
|
65
|
+
|
|
66
|
+
2. **Value map** — add to `RAW_PAIRS` (next to the capacity pairs):
|
|
67
|
+
```python
|
|
68
|
+
("cumulative_charge_energy", "charge_energy"),
|
|
69
|
+
("cumulative_discharge_energy", "discharge_energy"),
|
|
70
|
+
```
|
|
71
|
+
|
|
72
|
+
3. **Exception sets** — remove from the raw exceptions so totality stays
|
|
73
|
+
exact:
|
|
74
|
+
- `LEGACY_ONLY_RAW`: drop `"charge_energy"`, `"discharge_energy"`
|
|
75
|
+
- `NATIVE_ONLY_RAW`: drop `"cumulative_charge_energy"`,
|
|
76
|
+
`"cumulative_discharge_energy"`
|
|
77
|
+
|
|
78
|
+
4. **Attribute map** — add to `LEGACY_ATTR_TO_SCHEMA["raw"]`:
|
|
79
|
+
```python
|
|
80
|
+
"charge_energy_txt": "cumulative_charge_energy",
|
|
81
|
+
"discharge_energy_txt": "cumulative_discharge_energy",
|
|
82
|
+
```
|
|
83
|
+
and remove those attrs from `LEGACY_ATTR_UNMAPPED["raw"]`.
|
|
84
|
+
|
|
85
|
+
5. **Do not touch** step/cycle maps (`NATIVE_ONLY_STEP` still has
|
|
86
|
+
`charge_energy` / `discharge_energy` as *step-table* base signals — different
|
|
87
|
+
columns).
|
|
88
|
+
|
|
89
|
+
6. **Behavior:** `legacy_to_native_raw()` / bridge renames will translate
|
|
90
|
+
energy instead of leaving it as a passthrough extra. That is the intended
|
|
91
|
+
fix for native-schema lookup and `harmonize()`.
|
|
92
|
+
|
|
93
|
+
## Files to touch
|
|
94
|
+
|
|
95
|
+
| Path | Change |
|
|
96
|
+
|------|--------|
|
|
97
|
+
| `src/cellpycore/legacy/mapping.py` | `RAW_PAIRS` + attr table + raw exception sets |
|
|
98
|
+
| `tests/test_header_mapping.py` | Extend `test_known_translations` expected dict with the two energy renames |
|
|
99
|
+
| `.issueflows/04-designs-and-guides/` (short note) | Optional: "raw energy bridged like capacity (#139)" so D3 staleness is local |
|
|
100
|
+
|
|
101
|
+
## Test strategy
|
|
102
|
+
|
|
103
|
+
```bash
|
|
104
|
+
uv run pytest tests/test_header_mapping.py
|
|
105
|
+
uv run ruff check && uv run ruff format --check
|
|
106
|
+
```
|
|
107
|
+
|
|
108
|
+
Existing totality / attr-sync tests should go green once exceptions and pairs
|
|
109
|
+
move together. No new fixture goldens expected (energy was not in
|
|
110
|
+
`RAW_PAIRS` before; bridge goldens that omit energy columns stay unaffected).
|
|
111
|
+
|
|
112
|
+
## Open questions
|
|
113
|
+
|
|
114
|
+
1. **Energy reset convention = capacity?** **Recommended: yes** (spec + #42
|
|
115
|
+
already say so). Accepting this plan accepts that answer for the GitHub
|
|
116
|
+
issue ask.
|
|
117
|
+
2. **Update architecture-plan D3 in this PR?** **Recommended: no** — keep
|
|
118
|
+
core PR small; optional local design-guide note only. Say if you want the
|
|
119
|
+
architecture-plan edit bundled.
|
|
@@ -0,0 +1,18 @@
|
|
|
1
|
+
# Issue #139 status
|
|
2
|
+
|
|
3
|
+
- [x] Done
|
|
4
|
+
|
|
5
|
+
## What's done
|
|
6
|
+
|
|
7
|
+
- Plan accepted (energy reset = capacity convention).
|
|
8
|
+
- `RAW_PAIRS` + `LEGACY_ATTR_TO_SCHEMA["raw"]` energy entries added.
|
|
9
|
+
- Dropped energy from `LEGACY_ONLY_RAW` / `NATIVE_ONLY_RAW` /
|
|
10
|
+
`LEGACY_ATTR_UNMAPPED["raw"]`.
|
|
11
|
+
- `test_known_translations` updated.
|
|
12
|
+
- Design note: `.issueflows/04-designs-and-guides/legacy-raw-energy-mapping.md`.
|
|
13
|
+
- `uv run pytest` — 261 passed; ruff clean.
|
|
14
|
+
- Version bump `0.2.2` → `0.2.3`; `HISTORY.md` promoted.
|
|
15
|
+
|
|
16
|
+
## Remaining work
|
|
17
|
+
|
|
18
|
+
- None (ship via PR).
|
|
@@ -0,0 +1,15 @@
|
|
|
1
|
+
# Legacy bridge carries `test_id` (#136)
|
|
2
|
+
|
|
3
|
+
**Context.** `OldCellpyCellCore` used to drop `test_id` from outbound steps /
|
|
4
|
+
summary via fixed column orders + `NATIVE_ONLY_*` exceptions. That forced cellpy
|
|
5
|
+
#507 to re-stamp steps so campaign summaries could window per test.
|
|
6
|
+
|
|
7
|
+
**Decision.** Bridge `test_id` as an identity column on steps and summary when
|
|
8
|
+
present. Window `_add_legacy_summary_cruft` by `test_id`. `merge_data` /
|
|
9
|
+
`update_data` require native `config.Schema` (hard `TypeError` on `Headers*`).
|
|
10
|
+
|
|
11
|
+
**Alternatives.** Opt-in flag for summary `test_id` — rejected (additive is
|
|
12
|
+
enough). Full legacy-schema merge aliases — rejected (step/cycle attrs incomplete;
|
|
13
|
+
cellpy uses pandas merge).
|
|
14
|
+
|
|
15
|
+
**Refs.** cellpy #507 / #510 / #511; release after this → re-pin cellpy.
|
|
@@ -0,0 +1,24 @@
|
|
|
1
|
+
# Legacy raw energy mapping (#139)
|
|
2
|
+
|
|
3
|
+
**Context.** Raw `charge_energy` / `discharge_energy` used to sit in
|
|
4
|
+
`LEGACY_ONLY_RAW` (passthrough). Architecture-plan D3 still says that and
|
|
5
|
+
claims different reset semantics vs native cumulative energy.
|
|
6
|
+
|
|
7
|
+
**Decision.** Bridge them like capacity via `RAW_PAIRS` /
|
|
8
|
+
`LEGACY_ATTR_TO_SCHEMA["raw"]`:
|
|
9
|
+
|
|
10
|
+
| legacy value | native |
|
|
11
|
+
|---|---|
|
|
12
|
+
| `charge_energy` | `cumulative_charge_energy` |
|
|
13
|
+
| `discharge_energy` | `cumulative_discharge_energy` |
|
|
14
|
+
|
|
15
|
+
Reset convention matches capacity: cycle-cumulative, per direction
|
|
16
|
+
(`docs/specifications/harmonized-raw.md` Capacity convention). Wrong source
|
|
17
|
+
granularity → issue #42 normalizer, not leaving columns unmapped.
|
|
18
|
+
|
|
19
|
+
**Alternatives.** Keep passthrough (rejected — breaks `to_native()` /
|
|
20
|
+
`harmonize()` schema lookup). Synthesize energy from other signals (rejected —
|
|
21
|
+
rename only).
|
|
22
|
+
|
|
23
|
+
**Refs.** #139; cellpy #560; supersedes architecture-plan D3 for these two
|
|
24
|
+
columns only.
|
|
@@ -79,3 +79,13 @@ uv run pre-commit install
|
|
|
79
79
|
the public API.
|
|
80
80
|
- Per-step stat column names (`<signal>_<stat>`) are a fixed engine contract,
|
|
81
81
|
not schema-injected (issue #70).
|
|
82
|
+
|
|
83
|
+
## Release & version bump
|
|
84
|
+
|
|
85
|
+
Static version in ``pyproject.toml`` (``project.version``). Bump with::
|
|
86
|
+
|
|
87
|
+
uv version --bump patch|minor|major
|
|
88
|
+
|
|
89
|
+
Cut a GitHub release from ``main`` after merge (``release`` alias / see
|
|
90
|
+
``release-procedure.md``) so ``release.yml`` publishes ``cellpycore`` to PyPI.
|
|
91
|
+
|
|
@@ -4,6 +4,22 @@ All notable changes to this project will be documented in this file.
|
|
|
4
4
|
|
|
5
5
|
## [Unreleased]
|
|
6
6
|
|
|
7
|
+
## [0.2.3] - 2026-07-19
|
|
8
|
+
|
|
9
|
+
- Bridge legacy raw ``charge_energy`` / ``discharge_energy`` to
|
|
10
|
+
``cumulative_charge_energy`` / ``cumulative_discharge_energy`` via
|
|
11
|
+
``RAW_PAIRS`` and ``LEGACY_ATTR_TO_SCHEMA``, matching the capacity
|
|
12
|
+
cycle-cumulative convention. (#139)
|
|
13
|
+
|
|
14
|
+
## [0.2.2] - 2026-07-17
|
|
15
|
+
|
|
16
|
+
- Legacy bridge carries ``test_id`` through step and summary frames (identity
|
|
17
|
+
mapping) so campaign-merged multi-test objects keep per-test summary
|
|
18
|
+
windowing without a consumer re-stamp. ``_add_legacy_summary_cruft`` cumsums
|
|
19
|
+
are windowed per ``test_id`` when present. ``merge_data`` / ``update_data``
|
|
20
|
+
raise ``TypeError`` on legacy ``Headers*`` schemas (require native
|
|
21
|
+
``config.Schema``). Golden step/summary snapshots regenerated. (#136)
|
|
22
|
+
|
|
7
23
|
## [0.2.1] - 2026-07-15
|
|
8
24
|
|
|
9
25
|
- Fix `OldCellpyCellCore.make_core_summary` to pass `test_mode` into
|
|
@@ -297,15 +297,18 @@
|
|
|
297
297
|
"295": "Community 295",
|
|
298
298
|
"296": "Community 296",
|
|
299
299
|
"297": "Community 297",
|
|
300
|
+
"298": "Community 298",
|
|
300
301
|
"299": "Community 299",
|
|
301
302
|
"300": "Community 300",
|
|
302
303
|
"301": "Community 301",
|
|
303
304
|
"302": "Community 302",
|
|
305
|
+
"303": "Community 303",
|
|
304
306
|
"304": "Community 304",
|
|
305
307
|
"305": "Community 305",
|
|
306
308
|
"306": "Community 306",
|
|
307
309
|
"307": "Community 307",
|
|
308
310
|
"308": "Community 308",
|
|
311
|
+
"309": "Community 309",
|
|
309
312
|
"310": "Community 310",
|
|
310
313
|
"311": "Community 311",
|
|
311
314
|
"312": "Community 312",
|
|
@@ -313,13 +316,15 @@
|
|
|
313
316
|
"314": "Community 314",
|
|
314
317
|
"315": "Community 315",
|
|
315
318
|
"316": "Community 316",
|
|
319
|
+
"317": "Community 317",
|
|
316
320
|
"318": "Community 318",
|
|
317
321
|
"319": "Community 319",
|
|
318
322
|
"320": "Community 320",
|
|
323
|
+
"321": "Community 321",
|
|
319
324
|
"322": "Community 322",
|
|
320
|
-
"323": "Community 323",
|
|
321
325
|
"324": "Community 324",
|
|
322
326
|
"325": "Community 325",
|
|
327
|
+
"326": "Community 326",
|
|
323
328
|
"327": "Community 327",
|
|
324
329
|
"328": "Community 328",
|
|
325
330
|
"329": "Community 329",
|
|
@@ -337,5 +342,26 @@
|
|
|
337
342
|
"341": "Community 341",
|
|
338
343
|
"342": "Community 342",
|
|
339
344
|
"343": "Community 343",
|
|
340
|
-
"344": "Community 344"
|
|
345
|
+
"344": "Community 344",
|
|
346
|
+
"345": "Community 345",
|
|
347
|
+
"346": "Community 346",
|
|
348
|
+
"347": "Community 347",
|
|
349
|
+
"348": "Community 348",
|
|
350
|
+
"349": "Community 349",
|
|
351
|
+
"350": "Community 350",
|
|
352
|
+
"351": "Community 351",
|
|
353
|
+
"353": "Community 353",
|
|
354
|
+
"354": "Community 354",
|
|
355
|
+
"356": "Community 356",
|
|
356
|
+
"357": "Community 357",
|
|
357
|
+
"358": "Community 358",
|
|
358
|
+
"360": "Community 360",
|
|
359
|
+
"361": "Community 361",
|
|
360
|
+
"362": "Community 362",
|
|
361
|
+
"363": "Community 363",
|
|
362
|
+
"364": "Community 364",
|
|
363
|
+
"365": "Community 365",
|
|
364
|
+
"366": "Community 366",
|
|
365
|
+
"367": "Community 367",
|
|
366
|
+
"368": "Community 368"
|
|
341
367
|
}
|