Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

milestone: Vestige 0.3.0 #8

Merged
merged 33 commits into from
Dec 25, 2024
Merged

milestone: Vestige 0.3.0 #8

merged 33 commits into from
Dec 25, 2024

Conversation

ascpixi
Copy link
Owner

@ascpixi ascpixi commented Dec 25, 2024

Changelog:

  • added unit testing via Vitest,
  • note output visualization (keyboard/piano),
  • implemented chorus nodes,
  • implemented arpeggiator nodes,
  • added ping-pong (stereo) mode to delay nodes,
  • refactoring:
    • overhauled implementing serializers for nodes,
    • reusable hooks like useBoundState
  • bug fixes:
    • LFO frequency is now properly restored,
    • removed "handle not found" warnings from React Flow.

ascpixi added 30 commits December 23, 2024 00:43
@ascpixi
fix: remove weird invisible character from robots.txt
8552fd4
@ascpixi
fix: address Lighthouse concerns
40ef374
@ascpixi
feat: add ping-pong mode to delay nodes
87cc6c5
@ascpixi
fix: remove "handle not found" warnings in console
6f9d1f0
@ascpixi
chore: seperate internal node functions into "basis.ts"
ef851a0
@ascpixi
feat: import instead of fetch default sample sets
8db0407
@ascpixi
fix: also serialize polyphony on PentatonicMelodyNodes
12762bc
@ascpixi
fix: move sample set data into /src/assets/sample-sets
de56ec0
@ascpixi
feat: allow for unit testing
50c85b6
@ascpixi
chore: remove unused jsdom dependency
2ac59d0
@ascpixi
feat(ci): run unit tests w/ GitHub Actions
2831034
@ascpixi
fix: remove "E#" from MIDI notes (E# = F)
f2a1480 
oops!
@ascpixi
feat: add note generator visualizer
3525f0b
@ascpixi
feat: add chorus nodes
13cef1f
@ascpixi
fix: re-serialize project after MIDI note numbers change
73cb92b
@ascpixi
chore: update dependencies
2a70b62
@ascpixi
fix: prevent Vitest from creating empty screenshot dirs
649cfa3 
We run in headless mode, and thus we cannot even create screenshots in the first place
@ascpixi
chore: prefer direct initialization instead of constructors
78a59f3
@ascpixi
chore: replace more ctors with initializers
423ddb1
@ascpixi
fix: revert back to ctor for SynthAudioGenerator (dependency on this)
37efd33
@ascpixi
feat: add FlatNodeDataSerializer to reduce repetition
052c76e
@ascpixi
fix: release all notes when changing synth oscillators
de45574
@ascpixi
feat: implement arpeggiator nodes
ed3d891
@ascpixi
chore: remove repetition via const aliases
6b88cc2
@ascpixi
feat: introduce useBoundState
67b03fc
@ascpixi
feat: introduce useNoteGeneratorSync
f20ee57
@ascpixi
chore: remove unused import from App.tsx
107271d
@ascpixi
fix: correct formula for DOWN arp style
03a8bb8
@ascpixi
chore: don't log key events
03dc22d
@ascpixi
fix: properly restore speed in LFOs
d55a9c3
@ascpixi ascpixi marked this pull request as ready for review December 25, 2024 00:14
@ascpixi ascpixi merged commit 495dad4 into main Dec 25, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ascpixi