docs: commenti inline sul perché in core/induttivo/lambda_space#4
Draft
Type666firewall wants to merge 1 commit into
Draft
docs: commenti inline sul perché in core/induttivo/lambda_space#4Type666firewall wants to merge 1 commit into
Type666firewall wants to merge 1 commit into
Conversation
Il codice era già densamente commentato sul WHY (docstring di modulo e funzione, rationale delle scelte di design). Questa passata aggiunge SOLO i vuoti reali trovati — numeri magici o scelte non ovvie senza motivazione accanto — senza restatement di ciò che è già spiegato altrove: - core.py: perché _to_native esiste (numpy leaks non serializzabili), perché il match testuale argomento↔fallacia (nessun indice condiviso tra Argomento e Patologia, span da passaggi diversi), perché la soglia 0.95 sul warning di bilanciamento Θ/ऋ (euristica non calibrata). - lambda_space.py: perché frozenset per LAMBDA_RESH (immutabilità: Λ evolve a livello di codice, mai per mutazione a runtime), perché _audit_invariants gira eager a import time (fail-fast, non a metà run). - induttivo.py: perché lo sconto ×0.8 sulla confidence dei segnali deterministici reinterpretati come hit Trilemma (inferenza su un pattern nato per altro scopo, non rilevazione diretta), perché arsenale/trilemma restano fuori da `da_eseguire` (hanno blocchi dedicati, non sono saltati). epsilon.py verificato ma non toccato: già il più commentato dei quattro. Zero modifiche di logica — solo commenti, verificato con py_compile e le batterie no-ML esistenti (tutte verdi, invariate). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016MnNuFahrz482cFEqCEnoq
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Su richiesta: commentare i file più importanti per leggere meglio il codice —
core.py,epsilon.py,lambda_space.py,induttivo.py— con commenti inline solo sul perché, non su cosa fa il codice (già leggibile dai nomi).Nota onesta: questi quattro file erano già commentati in modo insolitamente denso (docstring di modulo/funzione, rationale delle scelte con tanto di data). Aggiungere spiegazioni ovunque avrebbe prodotto rumore, non chiarezza. Ho cercato i vuoti reali — numeri magici o scelte non ovvie senza motivazione accanto — e ne ho trovati pochi:
core.py: perché_to_nativeesiste (i tipi numpy che filtrano da NLI/embedding non sono JSON-serializzabili); perché il match testuale argomento↔fallacia con finestra+200(nessun indice condiviso traArgomento, dalle proposizioni segmentate, ePatologia, dagli span sul testo grezzo — passaggi diversi sulla stessa frase); perché la soglia0.95sul warning di bilanciamento Θ/ऋ (euristica, non calibrata — come i pesi diepsilon.py).lambda_space.py: perchéfrozensetperLAMBDA_RESH(immutabilità: Λ evolve a livello di codice/commit, mai per mutazione a runtime dell'agente); perché_audit_invariants()gira eager a import time (fail-fast: un invariante rotto ferma il modulo subito, non emerge a metà run).induttivo.py: perché lo sconto×0.8sulla confidence dei segnali deterministici reinterpretati come hit Trilemma (è un'inferenza su cosa implica un pattern nato per un altro scopo — NON_SEQUITUR/circolarità — non una rilevazione diretta del corno, quindi meno affidabile del match regex diretto); perchéarsenale/trilemmarestano fuori dada_eseguire(non sono saltati — hanno ciascuno un blocco dedicato più sotto, con payload/feed diversi dal generico).epsilon.py: verificato per intero, non toccato — è già il più commentato dei quattro (ogni peso ha data e motivo della sua ricalibrazione).Verifica
Zero modifiche di logica:
py_compilesu tutti e tre + le batterie no-ML esistenti (test_aggregatore,test_postprocess_inclosura,test_persistenza_doc,test_trilemma_predetect) — tutte verdi, comportamento invariato. Diff: solo righe aggiunte (26 righe, 3 file), nessuna riga di codice toccata.🤖 Generated with Claude Code
https://claude.ai/code/session_016MnNuFahrz482cFEqCEnoq
Generated by Claude Code