[Genabel-commits] r1705 - pkg/ProbABEL/src

L.C. Karssen lennart at karssen.org
Mon Apr 28 12:03:32 CEST 2014


Hi Maarten,

That's interesting. I assume you did this in response to bug #5658?
The change you made is only for ASCII input files, right? Any idea how
this is treated in GenABEL's mach2databel() and impute2databel()? I
assume the NAs are converted to IEEE754 compatible NaN there, but I'm
not sure. If that is the case, then this would fix that bug, right?


Lennart.

On 28-04-14 11:49, noreply at r-forge.r-project.org wrote:
> Author: maartenk
> Date: 2014-04-28 11:49:36 +0200 (Mon, 28 Apr 2014)
> New Revision: 1705
> 
> Modified:
>    pkg/ProbABEL/src/gendata.cpp
> Log:
> -detecting  not avaibles in mldose file more rigid(catches [Nn][NnAa]?)
> 
> Modified: pkg/ProbABEL/src/gendata.cpp
> ===================================================================
> --- pkg/ProbABEL/src/gendata.cpp	2014-04-27 22:17:28 UTC (rev 1704)
> +++ pkg/ProbABEL/src/gendata.cpp	2014-04-28 09:49:36 UTC (rev 1705)
> @@ -57,11 +57,11 @@
>  
>          // check NaN (right now checks only first character)
>          // TODO: make catching of NaN more rigid
> -        if (*all_numbers == 'N')
> +        if (toupper(*all_numbers) == 'N')
>          {
>              result = std::numeric_limits<double>::quiet_NaN();
>              // Skip other characters of NaN
> -            while ((*all_numbers == 'a') | (*all_numbers == 'N'))
> +            while ((toupper(*all_numbers) == 'A') | (toupper(*all_numbers) == 'N'))
>              {
>                  all_numbers++;
>              }
> 
> _______________________________________________
> Genabel-commits mailing list
> Genabel-commits at lists.r-forge.r-project.org
> https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/genabel-commits
> 

-- 
*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*
L.C. Karssen
Utrecht
The Netherlands

lennart at karssen.org
http://blog.karssen.org
GPG key ID: A88F554A
-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 213 bytes
Desc: OpenPGP digital signature
URL: <http://lists.r-forge.r-project.org/pipermail/genabel-commits/attachments/20140428/322f4695/attachment.sig>


More information about the Genabel-commits mailing list