-
Notifications
You must be signed in to change notification settings - Fork 13
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.
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
Append ts segments to m3u8 file #47
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
To be updated
* Mseed (orcasound#31) * Initial version for mseed and multiple architectures on single branch * fixed ffmpeg command * made files safe names * Hardcod URL for now, made env vars to int * removed from project * Fixed debug message * make data directory if it doesn't exist * initial combo version * initial version * run actual commands, add logspout * fix typos, use shutil.move instead of os.rename for docker * Fixed m3u8 path * fixed to encode in realtime, wait for file * change to ts files * use a bunch of ts files * force new dummy.ts on start * Fix Dockerfile case sensitive name * Added logging * make streaming privleged to use nice * fix audio sampling rate at 64000 * check before deleting/moving files, fixed audio rate, speed up start * change to mono * updates * Split docker-compose.yml into rpi and x86 version * Fix file permsissions * chagne segment to minutes, debug statements * make node run for 24 hours * Fixed rpi baseline build * added support for virtual streaming mode * moved to node folder * made different compose files for build and pull * Added info about logical soundcard names * checking .env adding hls loopback * add image tag for rpi * Removed empty build section To be updated * Fix typo in readme sentence. * Fix minor typos in README paragraph 1 Co-authored-by: steve <steve@[email protected]> Co-authored-by: Scott Veirs <[email protected]> * Add example .env file for node streaming development * Clarify intro; add emphasis re directory structure, Pisound link * Add Support for Arm64 (orcasound#33) * arm64: install ffmpeg instaed of building * No logspout for arm64 There is no logspout for arm64 so comment this out. * Create docker-compose.arm64-pull.yml New version to support arm64 dockerhub image Co-authored-by: JoyceLiao <[email protected]> * Bring README up to date, add links and fix broken ones. * typo fix on 'README.md' (Line 13). Fixed broken link by changing 'https://live.orcaound.net' to 'https://live.orcasound.net' (orcasound#37) * Fetch data using ooipy * remove conflict markers * add requirements to DockerFile * convert to .ts * generate .m3u8 manifest file Co-authored-by: mcshicks <[email protected]> Co-authored-by: steve <steve@[email protected]> Co-authored-by: Scott Veirs <[email protected]> Co-authored-by: JoyceLiao <[email protected]> Co-authored-by: Evan-Scallan <[email protected]>
* Mseed (orcasound#31) * Initial version for mseed and multiple architectures on single branch * fixed ffmpeg command * made files safe names * Hardcod URL for now, made env vars to int * removed from project * Fixed debug message * make data directory if it doesn't exist * initial combo version * initial version * run actual commands, add logspout * fix typos, use shutil.move instead of os.rename for docker * Fixed m3u8 path * fixed to encode in realtime, wait for file * change to ts files * use a bunch of ts files * force new dummy.ts on start * Fix Dockerfile case sensitive name * Added logging * make streaming privleged to use nice * fix audio sampling rate at 64000 * check before deleting/moving files, fixed audio rate, speed up start * change to mono * updates * Split docker-compose.yml into rpi and x86 version * Fix file permsissions * chagne segment to minutes, debug statements * make node run for 24 hours * Fixed rpi baseline build * added support for virtual streaming mode * moved to node folder * made different compose files for build and pull * Added info about logical soundcard names * checking .env adding hls loopback * add image tag for rpi * Removed empty build section To be updated * Fix typo in readme sentence. * Fix minor typos in README paragraph 1 Co-authored-by: steve <steve@[email protected]> Co-authored-by: Scott Veirs <[email protected]> * Add example .env file for node streaming development * Clarify intro; add emphasis re directory structure, Pisound link * Add Support for Arm64 (orcasound#33) * arm64: install ffmpeg instaed of building * No logspout for arm64 There is no logspout for arm64 so comment this out. * Create docker-compose.arm64-pull.yml New version to support arm64 dockerhub image Co-authored-by: JoyceLiao <[email protected]> * Bring README up to date, add links and fix broken ones. * typo fix on 'README.md' (Line 13). Fixed broken link by changing 'https://live.orcaound.net' to 'https://live.orcasound.net' (orcasound#37) * Fetch data using ooipy * remove conflict markers * add requirements to DockerFile * convert to .ts * generate .m3u8 manifest file * resolve shutil error Co-authored-by: mcshicks <[email protected]> Co-authored-by: steve <steve@[email protected]> Co-authored-by: Scott Veirs <[email protected]> Co-authored-by: JoyceLiao <[email protected]> Co-authored-by: Evan-Scallan <[email protected]>
mcshicks
approved these changes
Aug 6, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently, the m3u8 file is recreated for every new ts segment. The goal is to append the names of new ts segments to the old m3u8 file. #42
writeManifest() function handles the logic for writing the m3u8 file