Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Transfer/Store LZMA compressed video chunks #130

Closed
flavioribeiro opened this issue Nov 11, 2014 · 5 comments
Closed

Transfer/Store LZMA compressed video chunks #130

flavioribeiro opened this issue Nov 11, 2014 · 5 comments

Comments

@flavioribeiro
Copy link
Contributor

https://github.com/nmrugg/LZMA-JS

@flavioribeiro
Copy link
Contributor Author

had a promising result with https://github.com/pieroxy/lz-string:

>>> chunk.length
272728
>>> LZString.compress(chunk).length
104454

@flavioribeiro
Copy link
Contributor Author

Segment bitrate: 1.2Mbps, size in seconds: 5

 [DEBUG] Without LZString: 1154072, with LZString: 484184 p2phls.min.js:3
 [DEBUG] Without LZString: 1124240, with LZString: 471442 p2phls.min.js:3
 [DEBUG] Without LZString: 1217488, with LZString: 511350 p2phls.min.js:3
 [DEBUG] Without LZString: 1048288, with LZString: 440291 p2phls.min.js:3
 [DEBUG] Without LZString: 1118224, with LZString: 470966 p2phls.min.js:3
 [DEBUG] Without LZString: 1220248, with LZString: 511604 p2phls.min.js:3
 [DEBUG] Without LZString: 1069344, with LZString: 449353 p2phls.min.js:3
 [DEBUG] Without LZString: 1131760, with LZString: 476222 p2phls.min.js:3
 [DEBUG] Without LZString: 1125996, with LZString: 473367 p2phls.min.js:3

Experiencing stucks, need to put compress/decompress functions in a separate web worker. :feelsgood:

@flavioribeiro
Copy link
Contributor Author

@flavioribeiro
Copy link
Contributor Author

Since char size calculation is needed, we come back with this issue: rtc-io/rtc-bufferedchannel#4

I'll try to figure out an alternative to https://github.com/rtc-io/rtc-bufferedchannel/blob/master/index.js#L212

flavioribeiro referenced this issue in flavioribeiro/rtc-bufferedchannel Nov 12, 2014
@flavioribeiro
Copy link
Contributor Author

All this transcoding thing is adding a lot of complexity on the code. I think its better to edit cdn_requester in order to encode a compressed Base64 string instead of the current one. This way, a AS3 version of https://github.com/pieroxy/lz-string is needed. 😭😭😭😭

flavioribeiro added a commit that referenced this issue Nov 12, 2014
flavioribeiro added a commit that referenced this issue Nov 12, 2014
…rder to fit dataChannel (refs #130)"

This reverts commit d2750f7.
flavioribeiro added a commit that referenced this issue Nov 12, 2014
…pository (refs #130)"

This reverts commit dadfdcc.

Conflicts:
	src/lz-string.js
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant