7 Commits

Include renames

Author	SHA1	Message	Date
larsandClaude Sonnet 5	81eb14d75c	Move scripts/ to giant/tools/ (issues.md Issue 9) ... `scripts` was published as a top-level distribution package, colliding with one of the most generic names in the Python ecosystem and shadowable by a stray scripts/ dir on the portal machines' shared /work/lbogner. Move it under the giant namespace; the dwarf command name is unchanged, only the Python import path and file location move. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-13 10:31:47 +02:00
lars	55332db67a	Bump ruff line-length to 120 and reformat ... Rejoins lines that only wrapped because they exceeded the old 88-char limit; ruff check and the full test suite (725 passed) are unaffected.	2026-08-12 13:33:09 +02:00
lars	05d5dee606	Apply ruff format after merging phase2-secondary-prediction ... The merged proc_idx/proc_map plumbing wasn't run through ruff format before merging; reflow only, no logic changes.	2026-07-15 10:00:36 +02:00
larsandClaude Sonnet 5	028fa13b7b	Drop orphaned child tracks instead of nulling secondary targets ... A listed child_track_id can fail to match any first-step row (e.g. a secondary absorbed below the tracking threshold at birth). The parent->child left join in _add_secondary_attributes left these as nulls, which silently became NaN once the parquet round-tripped through the loader's float32 padding — poisoning every later secondary slot in that step via the cumulative "remaining budget" in encode_secondaries, while e_sec quietly undercounted and n_sec (from len(child_track_ids)) overcounted relative to the actual lists. Drop orphans from both the per-secondary lists and child_track_ids itself so downstream counts stay consistent, and thread the per-file orphaned count back through convert_steps_to_parquet so both the sequential and --jobs>1 batch paths in `dwarf convert` can report an aggregate total instead of relying on grepping printed output. Also floors encode_secondaries' slot-0 budget to _EPS (matching the i>0 branch), fixing a harmless but noisy 0/0 divide warning on zero-secondary steps. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-07-09 14:36:06 +02:00
larsandClaude Opus 4.8	4ba419ebe4	Merge energy-conservation-poc into phase2-secondary-prediction ... Brings the energy-conservation PoC work (dwarf CLI unification, dwarf status improvements, predict --comment, ODE-step comparison scripts, predict-parquet-only analysis refactor) onto the Phase 2 branch. Conflict resolution: - giant/analysis.py: took the energy-conservation-poc version wholesale. That branch deliberately removed the live checkpoint+sampler diagnostics path (ModelBundle/load_model_bundle/make_val_loader/collect_samples) in favor of reading `giant predict --coord local` parquet output. Phase 2's only edits to this file adapted the removed path to the new dataset API, so nothing Phase-2-specific is lost; no external code called those funcs. Fixes for pre-existing breakage surfaced by the merge (both predate it): - giant/cli.py: predict's `_process` unpacked build_features into 5 values, but Phase 2 made it return 8 (added n_sec/sec_cont/sec_pdg_idx). Expanded the unpack; `giant predict --coord local` would have crashed otherwise. - tests/test_steps_to_parquet.py: Phase 2 renamed _add_secondary_energy -> _add_secondary_attributes without updating this test. Renamed the calls and extended the fixture with the pdg/pre_d{x,y,z} columns the expanded function reads; e_sec assertions unchanged. - analysis/compare_ode_steps_energy_conservation.py: E731 lambda assignment (added in the un-linted final PoC commit) rewritten as a def. ruff, ty, and pytest (179 passed) all green. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>	2026-07-06 12:18:30 +02:00
larsandClaude Sonnet 4.6	5be91e0d17	Expose dataset/conversion scripts as uv entry points ... scripts/ is now a proper package (scripts/__init__.py, added to the wheel's packages), with each script registered under [project.scripts] using its bare dashed name (e.g. `uv run migrate-geant-steps`). Tests now import these modules normally instead of loading them by file path. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-25 16:51:47 +02:00
lars	8475199609	Encode edep/secondary/post energy as a conservation-constrained simplex ... Replaces the independent log_delta_e/log_edep targets with 2 additive-log-ratio coordinates over the deposit/secondary/post-energy simplex (fractions of pre_E summing to 1), so edep + e_sec + post_E == pre_E holds by construction after decoding (softmax) rather than being learned approximately. Requires e_sec (secondary energy) as a new conditioning input and a steps_to_parquet.py pass to derive it from child track first-step energies.	2026-06-25 16:01:13 +02:00