Skip to content

Cpal/input support #65

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
Draft
breezykermo wants to merge 8 commits into
base: next
Choose a base branch
from
Draft

Cpal/input support #65

breezykermo wants to merge 8 commits into from

Conversation

@breezykermo
Copy link
Collaborator

@breezykermo breezykermo commented Feb 15, 2025 •
edited
Loading

  • Finds a default input device via cpal's discovery mechanism
  • Wires input data into the elementary engine appropriately
  • Introduces tracing for better structured logging
  • Removes env_logger, as tracing supplants its functionality.
  • Introduces thiserror for structured error handling
  • Introduces ringbuf to pass data between input and output streams.
  • Introduces clap to expose and handle CLI arguments.

@breezykermo
Migrates to tracing ecosystem for logging
c948326 
Tracing and tracing subscriber are the idiomatic way to instrument Rust
programs with logs.
@breezykermo
Scaffolds better error handling
11f926c 
The thiserror crate will allow us to better handle errors as they are
emitted from various parts of the program. For now, I propose that we
just default to propagating all errors up to the level of
`ElementaryCliError`; but as the runtime gets more nuanced, we can
decide how to better handle non-critical errors.
@breezykermo
Combines futuers with try_join in event loop
9793de1 
If either the event poller or the TCP listener thread returns an error,
we propagate it up to the CLI.
@breezykermo
Cleans up elem lib; publicity and dead code
f4a902a
@breezykermo
Scaffolds input incorporation with ringbuf
d07fbb8
@breezykermo
Introduces clap for argument parsing; adds a --list-devices flag
07a4acc 
`--list-devices` lists all devics that are discoverable via cpal. If a
device has neither input nor output, then we don't list it.
@breezykermo
Adds Nix flake
e36bfc7 
As a development environment; but also potentially for CI down the line
@breezykermo
Adds flake.lock
55fecd1
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

@breezykermo