Code Monkey home page Code Monkey logo

Comments (5)

achaloyan avatar achaloyan commented on September 13, 2024
Thanks for the detailed description, Vali.

1. RTSP Transport
Actually RTSP Transport header can be specified only in RTSP SETUP requests/responses.
http://tools.ietf.org/html/rfc2326#page-58

So I'll just remove Transport header generation from the responses sent to RTSP
DESCRIBE requests.

2. Announcement of telephone-event support.
For now, I'm going to add telephone-event next to other hard-coded codecs which
construct the response to the resource discovery request.
Finally, actual resource discovery will be implemented in the scope of Issue-19.

Reported by achaloyan on 2009-08-17 16:36:17

  • Status changed: Started
  • Labels added: Type-Defect, Priority-Medium, OpSys-All, Component-Server, Component-MRCPv1

from unimrcp.

achaloyan avatar achaloyan commented on September 13, 2024
Thanks, GVP works fine now. I hope it did not break compatibility with other products
somehow.

By the way, we experienced another problem with GVP: NLSML result cannot contain any
white-space at the end of the message. Our ASR engine returns \r\n at the end so I
treated it in our plugin. This is just a note for those dealing with GVP, I would not
recommend to fix it in UniMRCP in general. It is definitely bug of GVP, because XML
specification allows white-space at the end of the document.

Reported by [email protected] on 2009-08-18 08:12:16

from unimrcp.

achaloyan avatar achaloyan commented on September 13, 2024
Well, actually it should not break anything, as I stated RTSP Trasnport header should
be present on in RTP SETUP. Yesterday I also checked the message flow with other MRCP
servers involved. Anyway, It'll be preferable if MRCPv1 specification (RFC4463)
contains at least one usage example, but it doesn't.

Thanks for sharing with us the NLSML issue you observed. I indeed see nothing, which
should be fixed in UniMRCP, at least other GVP users will be aware not to put any
additional white-spaces at the end of the doc.

So this issue is fixed. I remember about hard-coded capabilities and will address
them in the scope of Issue-19, when possible.

Reported by achaloyan on 2009-08-18 13:25:18

  • Status changed: Fixed

from unimrcp.

achaloyan avatar achaloyan commented on September 13, 2024
Hi Arsen,

I am responding to the Issues to verify challenge. Well, I think you can mark this
issue as verified. The DESCRIBE request now works fine (the rest is under Issue-19),
the thing with white-space in the end of NLSML is put here so that plugin developers
can find it and be aware of it.

The interoperability with GVP is still being tested so we might find another issue,
but I believe it would be problem of GVP so no patch to UniMRCP should be done and
the issue would be rather put on wiki.

But still I recall one minor issue observed with GVP. When we raise
RECOGNITION-COMPLETE event with completion cause
RECOGNIZER_COMPLETION_CAUSE_NO_MATCH_MAXTIME, GVP reports error and stops processing
the dialog. Because RECOGNIZER_COMPLETION_CAUSE_NO_MATCH_MAXTIME is introduced in
MRCPv2 spec, but GVP uses MRCPv1 (and RTSP). Shouldn't UniMRCP be aware of checking
such situations? In my opinion r1076 is adequate solution.

Vali

Reported by [email protected] on 2009-08-25 10:08:35

from unimrcp.

achaloyan avatar achaloyan commented on September 13, 2024
Hi Vali,

Thanks for the response.
You can continue the interoperability with GVP and report new issues if anything found.

I agree that sending MRCPv2 completion cause in the scope of MRCPv1 session is not
entirely correct. It should be easy to add an appropriate checking in the core to
allow only v1 causes. More over, it's possible to map additional causes introduced
in
v2 to v1. However, that causes are not fully backward compatible. I assume, the best
place to distinguish and construct appropriate messages is plugin itself. Therefore,
I added that ability in r1076 according to your suggestion. The rest is up to plugin
developers.

Reported by achaloyan on 2009-08-25 11:41:52

  • Status changed: Verified

from unimrcp.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.