Repository navigation
Conversation
La protezione dei segreti di GitHub scambia "Mistral3ForConditionalGeneration" (un nome di classe dentro transformers.min.js) per una chiave API di Mistral e blocca il push su gh-pages: la versione prova era rimasta alla 1.2.1. Su Pages si toglie dist/ai e la libreria arriva dalla CDN come prima; le app la tengono. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TqgA6T2ncfxMbmoaKeJWS4
pulisci() contava "non usato da un po'" con l'orologio: se un giro di disegno durava più di 1,5 s (computer lento, scheda video che arranca, pagina ferma) buttava il flusso che si stava guardando subito prima del disegno, e quello nuovo ripartiva dal fotogramma chiave (con il GOP lungo anche 20 s prima): immagine ferma per secondi, poi nera quando anche la ricerca veniva chiusa. Era la prova "play dal mezzo col GOP lungo" che falliva su main con la 1.2.1 e la 1.3.0. Ora si conta dall'ultima richiesta di un fotogramma (ultimaRichiesta), con un tetto di 6 s per i flussi e 20 s per le ricerche, e chi presta il fotogramma di ripiego resta vivo. Col processore rallentato 6x: prima 4 play su 6 fermi, ora 6 su 6 si muovono subito. Prova nuova che fallisce senza la correzione. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TqgA6T2ncfxMbmoaKeJWS4
This branch has not been deployed
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
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.
Due correzioni trovate guardando la CI su
maindopo l'uscita della 1.3.0.1. Il play si fermava (e andava nero) sui computer lenti
Su
main, sia con la 1.2.1 sia con la 1.3.0, falliva la prova "play dal mezzo di una ripresa col GOP lungo: il video si muove" ({"nati":3,"posizioni":[40,40,null,11,null]}): immagine ferma, poi nessuna immagine. Sulle PR passava, quindi dipendeva dalla velocità del runner.La causa (
src/media/fotogrammi.ts):pulisci()contava "non usato da un po'" con l'orologio. Durante il play il motore lo chiama a ogni giro prima di disegnare: se il giro precedente era durato più di 1,5 s (computer lento, scheda video che arranca, lettura dei pixel nella prova), il flusso che si stava guardando veniva buttato e quello nuovo ripartiva dal fotogramma chiave (qui 20 s di GOP). Nel frattempo si vedeva il fotogramma della ricerca, finché dopo 8 s anche la ricerca veniva chiusa: nero. L'ho visto coi log, col processore di Chromium rallentato 6× (CDPsetCPUThrottlingRate):nuovo {cursore:true}→ 2,2 s dopopulisci flusso→nuovo {cursore:false}.La correzione: si conta dall'ultima richiesta di un fotogramma (
ultimaRichiesta). Un flusso è vecchio se altri fotogrammi si sono chiesti per 1,5 s senza di lui, non se il disegno è lento; tetto di 6 s per i flussi e 20 s per le ricerche (vuoti in timeline, pagina ferma a lungo). Chi presta il fotogramma di ripiego (ricerca, altra qualità) resta vivo finché serve.Prima/dopo, 6 play a 6× più lento: prima 4 su 6 fermi per secondi (uno nero),
nati2; dopo 6 su 6 si muovono da subito,nati1.Prova nuova (in
test/prove.mjs, sezione "ripresa coi fotogrammi chiave radi"): la pagina ferma 1,8 s non fa buttare il flusso che si guarda, e uno che non serve più (si disegna un'altra clip) si chiude ancora. Senza la correzione fallisce (dopo:false, nati:1), con la correzione passa.__dpvTest.FTespone il modulo dei fotogrammi.2. Pages non si aggiornava
Dopo l'unione della 1.3.0 il push su
gh-pagesè stato bloccato dalla protezione dei segreti: inai/transformers.min.jsha visto una "Mistral AI API Key", che è il nome di classeMistral3ForConditionalGeneration(falso allarme).pages.ymlora togliedist/aiprima del push: sul sito di prova la libreria arriva dalla CDN come nella 1.2.1 (libreria()ci passa da sola), le app la tengono dentro. In alternativa si può segnare il falso allarme come consentito nelle impostazioni di sicurezza del repository: è una scelta tua.Versione
1.3.1 in
package.json, lock,Cargo.toml/Cargo.lock; voce nelCHANGELOG.md; note inCLAUDE.md.Prove
npx tsc --noEmitpulito; la prova nuova passa con la correzione e fallisce senza; 6 giri a CPU rallentata come sopra.node test/prove.mjsin corso in locale (e qui in CI).🤖 Generated with Claude Code
https://claude.ai/code/session_01TqgA6T2ncfxMbmoaKeJWS4