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

Create server #1

Merged
merged 16 commits into from
Nov 4, 2024
Merged

Create server #1

merged 16 commits into from
Nov 4, 2024

Conversation

Noarkhh
Copy link
Collaborator

@Noarkhh Noarkhh commented Sep 23, 2024

No description provided.

@Noarkhh Noarkhh requested a review from mat-hek September 23, 2024 14:03
Copy link

@mat-hek mat-hek left a comment

Choose a reason for hiding this comment

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

  • please update the readme
  • add a usage example there - can be sending an mp4 via rtsp to Boombox
  • you can also make it a test - run a server, send a file to Boombox, compare it with a fixture

BTW, I enabled Circle for this repo - it should run with the next commit

end

defp build_tail(builder, type, config) do
config |> IO.inspect(label: type)
Copy link

Choose a reason for hiding this comment

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

leftover

lib/membrane/simple_rtsp_server.ex Outdated Show resolved Hide resolved
@Noarkhh
Copy link
Collaborator Author

Noarkhh commented Sep 25, 2024

The test should pass once Boombox doesn't transcode when the input and output codecs are different

Process.sleep(50)
Boombox.run(input: "rtsp://localhost:40001/", output: output_path)

assert File.read!(input_path) == File.read!(output_path)
Copy link

Choose a reason for hiding this comment

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

For now, you can keep the output reference fixture and compare to it

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It seems that each time the output mp4 is generated it's slightly different

dupa.exs Outdated
Copy link

Choose a reason for hiding this comment

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

perform deduping

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

whoops, a careless git add :P

@@ -99,7 +95,11 @@ defmodule Membrane.SimpleRTSPServer.Pipeline do
clock_rate: config.clock_rate
]
)
|> child({:realtimer, type}, Membrane.Realtimer)
|> then(
&if realtime,
Copy link

Choose a reason for hiding this comment

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

do you have a use case for realtime: false? I think it would overload the socket buffer and cause packet loss

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I wanted to have the test not take so long, but came to the same conclusion

Copy link

Choose a reason for hiding this comment

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

An alternative would be an option in realtimer that would speed it up a few times. And a relatively short fixture - I'm using up to 10s for such cases

@Noarkhh Noarkhh requested a review from mat-hek September 30, 2024 09:04
@Noarkhh Noarkhh merged commit 39b0621 into master Nov 4, 2024
3 checks passed
@Noarkhh Noarkhh deleted the create-server branch November 4, 2024 14:42
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.

2 participants