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

Finalize Libraries to be added into main #9

Open
TheJollyDuck opened this issue Nov 11, 2023 · 1 comment
Open

Finalize Libraries to be added into main #9

TheJollyDuck opened this issue Nov 11, 2023 · 1 comment
Assignees
@TheJollyDuck
Labels
feature Something new to be added to the project formatting improvement Refactoring code to improve clarity and/or efficiency

Comments

@TheJollyDuck
Copy link
Owner

Currently, the libraries that is used on my personal machine are not included in the repository as I need some time to isolate and document the updating process with the project itself.

Once completed, it will then be push to remote.
@TheJollyDuck TheJollyDuck added feature Something new to be added to the project formatting improvement Refactoring code to improve clarity and/or efficiency labels Nov 11, 2023
@TheJollyDuck TheJollyDuck self-assigned this Nov 11, 2023
TheJollyDuck added a commit that referenced this issue Nov 11, 2023
@TheJollyDuck
Finalized some libraries for #9 + commented debug code.
b30d542
TheJollyDuck added a commit that referenced this issue Nov 11, 2023
@TheJollyDuck
Added CompiledNN lib + Updated README.MD #7 #9
1fe6117 
[Added]
- CompiledNN manually compiled and added to the lib directory.
- Base code to test linking and inclusion of new libraries are correct.

[Changed]
- README.MD better reflects the new dependency changes and clarifies
  better how to compile and licensing.
- Makefile now includes linking instructions for protobuf and
  CompiledNN.

[TODO]
- Expand on the Whistle Detection and FFTs.
- Get a working audio stream to test these features.

[Comments]
- CompiledNN was one of the most annoying and hardest library inclusion
  to this project so far. Maybe TFLite was the better option.
@TheJollyDuck
Copy link
Owner Author

The last thing we need to check is if we want to stick with miniaudio for both playback and r/w. Once we get code that works for both, and implement lua inclusion, this issue is ready to close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TheJollyDuck TheJollyDuck

Labels
feature Something new to be added to the project formatting improvement Refactoring code to improve clarity and/or efficiency
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@TheJollyDuck