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

opus-wav conversion for core utility (voip focus) #154

Closed
getnamo opened this issue Sep 5, 2019 · 5 comments
Closed

opus-wav conversion for core utility (voip focus) #154

getnamo opened this issue Sep 5, 2019 · 5 comments
Labels
addressed awaiting release enhancement

Comments

@getnamo
Copy link
Owner

getnamo commented Sep 5, 2019

No description provided.

@getnamo
Copy link
Owner Author

getnamo commented Sep 5, 2019

work tracked in https://github.com/getnamo/socketio-client-ue4/tree/feature-opus

@getnamo
Copy link
Owner Author

getnamo commented Sep 9, 2019 •
edited
Loading

non-file opus (minimal serialized format) compression works, oggs-file wrapper remains.

Minimal Serialized format is defined as (updated 15/09/19):

int32 -> Number of Packets
TArray<int16> Packet Size Array
TArray<int8> CompressedBytes

example serialization: https://github.com/getnamo/socketio-client-ue4/blob/763a855af49a426822c9b832a6604fb78c72eba8/Source/CoreUtility/Private/CUOpusCoder.cpp#L180

@getnamo
Copy link
Owner Author

getnamo commented Sep 12, 2019

opus oggs support deferred to later date

@getnamo
Copy link
Owner Author

getnamo commented Sep 12, 2019

Released in https://github.com/getnamo/socketio-client-ue4/releases/tag/v1.1.0

@getnamo getnamo closed this as completed Sep 12, 2019
@getnamo
Copy link
Owner Author

getnamo commented Sep 15, 2019

With 350d882 we've changed packet size array to int16 type to save ~1-2% of overhead

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
addressed awaiting release enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@getnamo