Kiax now ready to go in?

[Emil Stoyanov]

George Danchev wrote:

>On Wednesday 17 May 2006 00:43, Emil Stoyanov wrote:
>  
>
>>Hi,
>>
>>Santiago Garcia Mantinan wrote:
>>    
>>
>>>>Iaxclient devel team is aware of Tipic improvements, however I'm not
>>>>sure why tipic stuff is not merged into the iaxclient cvs (at least for
>>>>the last few weeks, I was out of town for quite a long time and havent
>>>>followed the mailing lists). I chose it because of the better experience
>>>>with EC (compared to the broken speex-based EC in iaxclient).
>>>>        
>>>>
>>>Umm... I wonder if somebody here knows about EC on iaxclient.
>>>
>>>      
>>>
>>>>>I'd really like to see upstream or at least Debian version of iaxclient
>>>>>with EC corrected, is there anything we can do to help this happen?
>>>>>          
>>>>>
>>>>That would be perfect, as I dont want to follow and cross-merge
>>>>iaxclient code from two sources. I will try to catch up these days with
>>>>the mailing list and will let you know if there is something done on EC
>>>>in standard iaxclient.
>>>>        
>>>>
>>>That would be great
>>>      
>>>
>>Well, I did take a look at the mailing list and commit history in the
>>SVN of iaxclient. For the last 3 moths there no radical changes, the EC
>>is the same (someone comlained recently from crash when EC enabled).
>>
>>    
>>
>>>>Actually.. I wondered. How do you decide when to build a new iaxclient0
>>>>package? Do you do your own versioning?
>>>>        
>>>>
>>>Well, I'm not the one taking care about iaxclient, maybe somebody else on
>>>the list (I'm Ccing it) can tell you more on this, but we are right now
>>>with iaxclient 0.0+cvs20050725 which seems to say that we got stuck with
>>>an iaxclient nearly one year old.
>>>      
>>>
>>OK, at least it is with the "new" API (better registration handling).
>>Since then there are some changes (suposing reflection in the api
>>header)
>>(http://svn.sourceforge.net/viewcvs.cgi/iaxclient/trunk/iaxclient/lib/iaxcl
>>ient.h?view=log). The current version of iaxclient in kiax is snapshot from
>>February and merged changes from Tipic's EC.
>>Please, in case you decide to ship kiax with the "original" iaxclient,
>>make the EC option disabled in src/mainwindow.ui, or inform me to give
>>you a changed one ;)
>>    
>>
>
>I suppose you are talking about removing Echo Cancellation from 
>src/prefswindow.ui ? 
>
>kiax-0.8.5-src$ diff -Naur src/prefswindow.ui~ src/prefswindow.ui
>--- src/prefswindow.ui~ 2006-05-17 19:13:38.286322680 +0300
>+++ src/prefswindow.ui  2006-05-17 19:13:38.328315394 +0300
>@@ -476,14 +476,6 @@
>                                     <bool>true</bool>
>                                 </property>
>                             </widget>
>-                            <widget class="QCheckBox" row="1" column="0">
>-                                <property name="name">
>-                                    
><cstring>echoCancelationCheckBox</cstring>
>-                                </property>
>-                                <property name="text">
>-                                    <string>Echo Cancelation</string>
>-                                </property>
>-                            </widget>
>                             <widget class="QCheckBox" row="0" column="1">
>                                 <property name="name">
>                                     <cstring>aagcCheckBox</cstring>
>
>
>pkg-voip team: how to approach iaxclient upstream to have a look at that Tipic 
>patch or we should patch our iaxclient package independantly ?. I'm not 
>falimiar with it, but probably it is the "Right Thing" [tm] to go ?
>
>  
>
The license of the implemented NLMS algorithm is very unclear. In
headers there is only:

 *
 * Copyright (C) DFS Deutsche Flugsicherung (2004, 2005).
 * All Rights Reserved.
 * Author: Andre Adrian

There is nothing pointing or noting what is the license.
Although it is packaged inside the library that Tipic ships and falls
under LGPL, is good idea to verify this, or just use the normal
iaxclient with disabled EC option.

Regards,
Emil