Skip to content

Conversation

@iammarco11
Copy link
Contributor

Feel free to leave suggestions for the readme imporvements

README.md Outdated
@@ -0,0 +1,32 @@
## <u>Progressive Streaming</u>

A simple application that allows the tranfer of a video file from the server to the client and upon download, has playback. Bonus addition of a transcoder script as well. Ffmpeg libav API is used for the decoding and transcoding the video file.
Copy link
Member

Choose a reason for hiding this comment

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

tranfer

Transfer*

transfer of a video file from the server to the client and upon download has playback.

Though not a review for the readme itself, but rather a note on the project itself: The main motive should be to download and playback the video simultaneously. Design an event loop whereas the buffer fill event occurs, the decoder should feed it to itself.

Bonus addition of a transcoder..

This can be another small section of the readme, just as another sentence would probably look neat.

Ffmpeg libav API is used for the decoding and transcoding the video file.

Libs used could be another section

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry for the spelling error, will fix everything

Copy link
Contributor Author

@iammarco11 iammarco11 Aug 13, 2020

Choose a reason for hiding this comment

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

Though not a review for the readme itself, but rather a note on the project itself: The main motive should be to download and playback the video simultaneously. Design an event loop whereas the buffer fill event occurs, the decoder should feed it to itself.

Yes I will work on that issue #2 soon.

A simple application that allows the tranfer of a video file from the server to the client and upon download, has playback. Bonus addition of a transcoder script as well. Ffmpeg libav API is used for the decoding and transcoding the video file.

### To run the script
Make sure you have a directory named bin in your project home to run the following commands.<br>
Copy link
Member

Choose a reason for hiding this comment

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

This could be automated, through the makefile no need to make the dev go through a simple hoop.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah once the makefile is done I will remove those, is that fine?

Copy link
Member

Choose a reason for hiding this comment

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

yeah

@iammarco11
Copy link
Contributor Author

when merged it fixes #4

@Aniketh01
Copy link
Member

Looks good for initial work. Would have to rework upon this. You can merge once you are ready to do so!

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.

3 participants