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.

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 61 commits into from
Aug 6, 2022
Merged

Conversation

karan2704
Copy link
Contributor

@karan2704 karan2704 commented Jul 13, 2022

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

mcshicks and others added 25 commits May 15, 2021 11:33
* 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]>
@karan2704 karan2704 marked this pull request as ready for review July 21, 2022 18:04
@mcshicks mcshicks self-requested a review August 6, 2022 13:37
Copy link
Contributor

@mcshicks mcshicks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks ok

@mcshicks mcshicks merged commit 9c71d60 into orcasound:master Aug 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants