Skip to content

Audio sync#3

Open
mirconegri wants to merge 8 commits into
InnovationTeamNest:masterfrom
mirconegri:master
Open

Audio sync#3
mirconegri wants to merge 8 commits into
InnovationTeamNest:masterfrom
mirconegri:master

Conversation

@mirconegri

Copy link
Copy Markdown

No description provided.

@elblasco elblasco self-requested a review March 6, 2026 12:08
@elblasco

elblasco commented Mar 6, 2026

Copy link
Copy Markdown
Member

Ciao, grazie mille della PR. Potresti incorporare le modifiche all'interno del file pattern-drawer.html? Ora do un'occhiata al codice.

Nota a margine, per i commit potresti inserire dei messaggi più chiari di cosa hai modificato in quel commit? Qualcosa tipo questo.

@mirconegri

Copy link
Copy Markdown
Author

Ciao, grazie mille della PR. Potresti incorporare le modifiche all'interno del file pattern-drawer.html? Ora do un'occhiata al codice.

Nota a margine, per i commit potresti inserire dei messaggi più chiari di cosa hai modificato in quel commit? Qualcosa tipo questo.

Ciao, questo è il file che abbiamo usato alla festa finale che aggiunge le foto prese dalla cartella corrente. Poi c'è anche l'opzione di regolare i bpm in caso i permessi di audio vengano negati.
Pensavo fosse una bella idea mettere una versione in più in un file separato. Fammi sapere, ciao!

Comment thread pattern-drawer_audio_image_sync.html
Comment thread pattern-drawer_audio_image_sync.html
Comment thread pattern-drawer_audio_image_sync.html Outdated
Comment thread pattern-drawer_audio_image_sync.html Outdated
Comment thread pattern-drawer_audio_image_sync.html
mirconegri and others added 6 commits March 6, 2026 17:59
@elblasco

elblasco commented Mar 8, 2026

Copy link
Copy Markdown
Member

Ora mi pare fiunzioni tutto; ma il mio messaggio è che non vorrei la feature delle immagini al centro, anche perché non è opt-in (o opt-out) ma sempre attiva. Poi il fatto di hard-codare il nome delle immagini con image<numero>.png non penso sia la soluzione ottimale.

- Removed `logo`, `logoIndex`, and `logoLoaded` state from constructor.
- Deleted `loadNextLogo()` and `renderLogoOverlay()` methods.
- Removed `drawImage` operation from the `requestAnimationFrame` loop.
- Eliminated Image object instantiations to reduce Garbage Collector pressure and heap memory allocations.
- Improved rendering performance by stripping expensive composite operations (`shadowBlur`, `globalAlpha` on images) from the main animation frame.
@mirconegri mirconegri changed the title Add image & audio sync Audio sync Mar 9, 2026
@elblasco

elblasco commented Mar 9, 2026

Copy link
Copy Markdown
Member

Ok, ora mi pare vada tutto.

Come ultime due cose ti chiederei di spostare tutte le modifiche sul file pattern_drawer.html, in un unico commit; ed infine di eliminare dalla history il file pattern-drawer_audio_image_sync.html.

Ah per implementare le features nel file principale basta che tu sovrascriva pattern-drawer.html con il comando mv attern-drawer_audio_image_sync.html pattern-drawer.html, da una shell linux.

Ultimissima cosa, una volta che hai fatto quanto scritto sopra, riusciresti a fare unoi squash dei commits ed a inserirci un messaggi in cui dici tipo feat: audio sync and bpm?

@mirconegri mirconegri closed this Mar 9, 2026
@elblasco elblasco reopened this Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants