gophone icon indicating copy to clipboard operation
gophone copied to clipboard

Documentation Issue Re Media

Open Pcrossley69 opened this issue 1 year ago • 10 comments

The help file doesn't indicate whether the media file is for playing or recording recording media It would be helpful if both input and output media were supported

There is also no indication of media types supported - can it plat/record wav ?

P.S. Thanks for this tool - I have been looking for a replacement for Sipcmd2 and this looks ideal.

Pcrossley69 avatar Mar 19 '25 19:03 Pcrossley69

It can play only wav, but it must match same sampling rate as codec used. Tool does only encoding here.

Thanks for sharing thoughts.

emiago avatar Mar 19 '25 21:03 emiago

Thanks for the quick response. I have a WAV file which works with sipcmd2 using G711a. It doesnt work with gophone

I'm using media=Filename

But it isn't playing - is my syntax incorrect?

Feature request:

I'm a trainer and try to use demonstrations whenever possible when teaching SIP.

I'd like to play a message then record a second message and play it back in the same session. I can do this using Sipcmd2 but noone seems to be developing that now and it has some limitations which gophone seems to improve upon.

I see from your timeline that recording is a feature you are working on so I thought I'd mention it.

Thanks again for your quick reply

Pcrossley69 avatar Mar 19 '25 22:03 Pcrossley69

@Pcrossley69 if you can send me file I can look. Recording will be added, yes. There is already code, just not my focus to finalize this.

emiago avatar Mar 19 '25 23:03 emiago

My audio files are ok with Sipcmd2 This is the spec

WAV file requirements:

  • mono
  • 8 kHz sampling rate
  • 16 bits sample size

On Wed, 19 Mar 2025, 23:21 Emia, @.***> wrote:

@Pcrossley69 https://github.com/Pcrossley69 if you can send me file I can look. Recording will be added, yes. There is already code, just not my focus to finalize this.

— Reply to this email directly, view it on GitHub https://github.com/emiago/gophone/issues/10#issuecomment-2738498191, or unsubscribe https://github.com/notifications/unsubscribe-auth/AU7HC72AIUTHNPUP76RKEET2VH3X7AVCNFSM6AAAAABZLWASNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMZYGQ4TQMJZGE . You are receiving this because you were mentioned.Message ID: @.***> [image: emiago]emiago left a comment (emiago/gophone#10) https://github.com/emiago/gophone/issues/10#issuecomment-2738498191

@Pcrossley69 https://github.com/Pcrossley69 if you can send me file I can look. Recording will be added, yes. There is already code, just not my focus to finalize this.

— Reply to this email directly, view it on GitHub https://github.com/emiago/gophone/issues/10#issuecomment-2738498191, or unsubscribe https://github.com/notifications/unsubscribe-auth/AU7HC72AIUTHNPUP76RKEET2VH3X7AVCNFSM6AAAAABZLWASNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMZYGQ4TQMJZGE . You are receiving this because you were mentioned.Message ID: @.***>

Pcrossley69 avatar Mar 20 '25 18:03 Pcrossley69

Aah, syntax is media=file=filename ;)

emiago avatar Mar 20 '25 21:03 emiago

Thanks for that. I did start the thread as a suspected documentation issue.

I notice that your examples include piping commands to stdin. Have you got a full list of accepted commands?

On Thu, 20 Mar 2025, 21:40 Emia, @.***> wrote:

Aah, syntax is media=file=filename ;)

— Reply to this email directly, view it on GitHub https://github.com/emiago/gophone/issues/10#issuecomment-2741731968, or unsubscribe https://github.com/notifications/unsubscribe-auth/AU7HC7YYFKTTI77QYWHJG3T2VMYUTAVCNFSM6AAAAABZLWASNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONBRG4ZTCOJWHA . You are receiving this because you were mentioned.Message ID: @.***> [image: emiago]emiago left a comment (emiago/gophone#10) https://github.com/emiago/gophone/issues/10#issuecomment-2741731968

Aah, syntax is media=file=filename ;)

— Reply to this email directly, view it on GitHub https://github.com/emiago/gophone/issues/10#issuecomment-2741731968, or unsubscribe https://github.com/notifications/unsubscribe-auth/AU7HC7YYFKTTI77QYWHJG3T2VMYUTAVCNFSM6AAAAABZLWASNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONBRG4ZTCOJWHA . You are receiving this because you were mentioned.Message ID: @.***>

Pcrossley69 avatar Mar 21 '25 06:03 Pcrossley69

Hi @Pcrossley69 Only what is provided as example for now. Transfers, call on hold, forwarding, start/stop recording could be added, but this requires dev time. My guess this is what are you aiming, but for now I had no need for this.

emiago avatar Mar 22 '25 09:03 emiago

Thanks for that.

Ive now got the audio file to play. It would however be nice if I could get it to play in a loop.

If you do update the documentation perhaps you could stress that audio files require a .wav extension.

Thanks again

On Sat, 22 Mar 2025, 09:06 Emia, @.***> wrote:

Hi @Pcrossley69 https://github.com/Pcrossley69 Only what is provided as example for now. Transfers, call on hold, forwarding, start/stop recording could be added, but this requires dev time. My guess this is what are you aiming, but for now I had no need for this.

— Reply to this email directly, view it on GitHub https://github.com/emiago/gophone/issues/10#issuecomment-2745152678, or unsubscribe https://github.com/notifications/unsubscribe-auth/AU7HC76VJP7VHQDWKEHEDS32VURX7AVCNFSM6AAAAABZLWASNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONBVGE2TENRXHA . You are receiving this because you were mentioned.Message ID: @.***> [image: emiago]emiago left a comment (emiago/gophone#10) https://github.com/emiago/gophone/issues/10#issuecomment-2745152678

Hi @Pcrossley69 https://github.com/Pcrossley69 Only what is provided as example for now. Transfers, call on hold, forwarding, start/stop recording could be added, but this requires dev time. My guess this is what are you aiming, but for now I had no need for this.

— Reply to this email directly, view it on GitHub https://github.com/emiago/gophone/issues/10#issuecomment-2745152678, or unsubscribe https://github.com/notifications/unsubscribe-auth/AU7HC76VJP7VHQDWKEHEDS32VURX7AVCNFSM6AAAAABZLWASNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONBVGE2TENRXHA . You are receiving this because you were mentioned.Message ID: @.***>

Pcrossley69 avatar Mar 22 '25 09:03 Pcrossley69

Feel free to open any issue or feature you would like. Documentation yes it is a must. I will leave this open to add that. I think playing files could be done better, and we need to split input/output controls at some point. Still I want users not to deal with complex flagging, when use cases are low.

emiago avatar Mar 22 '25 10:03 emiago

I've opened a few feature requests as you suggested.
Thanks again, I think this could be a really useful teaching aid, particularly because of the way you display the log messages.

Pcrossley69 avatar Mar 24 '25 12:03 Pcrossley69