r - Operator == inconsistent in logical columns in data.table -

-

please see following reproducible example:

library(data.table) set.seed(123) dt <- data.table(a=rep(0.3,10000)) dt[, b := runif(.n) < a] dt[b == t, .n] # [1] 3005 dt[, summary(b)] #    mode   false    true    na's # logical    6995    3005       0

everything looks fine , count of "true" values same 2 methods. replace col b new one. 

dt[, b := runif(.n) < a] dt[b == t, .n] # [1] 3331 dt[, summary(b)] #    mode   false    true    na's # logical    6981    3019       0

the count of 't' in column b different!!! same column 1 method gives 3331 "true" values , other 3019.

when == bypassed

dt[b != f, .n] # [1] 3019 dt[, summary(b)] #    mode   false    true    na's # logical    6981    3019       0

which correct again

i can reproduce data.table v1.94 , 1.9.5 on windows 8.1 x64.

here's easier reproducible example without runif().

require(data.table) ## 1.9.4+ dt = data.table(x = 1:5) dt[, y := x <= 2l] #    x     y # 1: 1  true # 2: 2  true # 3: 3 false # 4: 4 false # 5: 5 false  dt[y == true, .n] # [1] 2             <~~~~~~ correct result.  dt[, y := x <= 3l] #    x     y # 1: 1  true # 2: 2  true # 3: 3  true # 4: 4 false # 5: 5 false  dt[y == true, .n] # [1] 2             <~~~~~~ incorrect result, should 3!

now fixed in v1.9.5 on github.

:= , set* drop secondary keys (new in v1.9.4) dt[x==y] works again after := or set* without needing options(datatable.auto.index=false). setkey() dropping secondary keys correctly. 23 tests added. user36312 reporting, #885.


Comments

Post a Comment

Popular posts from this blog

python - pip install -U PySide error -

-
does know how avoid following error when running pip install -u pyside per official website instructions here: https://pypi.python.org/pypi/pyside/#installing-pyside-on-a-mac-os-x-system note have done brew install qt successfully. you using pip version 7.0.3, version 7.1.2 available. should consider upgrading via 'pip install --upgrade pip' command. collecting pyside using cached pyside-1.2.4.tar.gz installing collected packages: pyside running setup.py install pyside complete output command /applications/anaconda/bin/python -c "import setuptools, tokenize;__file__='/private/var/folders/dy/ttgqhqqx3g9bsbsnqnxsk2z80000gq/t/pip-build-cpxemt/pyside/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /var/folders/dy/ttgqhqqx3g9bsbsnqnxsk2z80000gq/t/pip-0r7hri-record/install-record.txt --single-version-externally-managed --compile: ...
Read more

arrays - C++ error: a brace-enclosed initializer is not allowed here before ‘{’ token -

-
simple question: i'm trying initialize array within c++ class declaration: using namespace std; #include <string> class myclass{ public: string myarray[] = {"a","b","c"}; }; and i'm getting error: error: brace-enclosed initializer not allowed here before ‘{’ token no, without complied c++11 compiler, cannot initialize member array in declaration. have initialize array member in constructor. , don't use open array if know number of elements initialize array.
Read more

cytoscape.js - How to add nodes to Dagre layout with Cytoscape -

-
i trying add nodes dagre graph through cityscape.js. problem nodes not painted. code have: cy.add({ group: "nodes", data: { id:"n0",name:"n0",weight: 75 }, }); $.ajax(url).done(function(jsontree){ var newnodes=[]; for(var i=1;i<6;i++){ //var child = jsontree.result[i]; newnodes.push( { group: "nodes", data: { id: "n"+i, name:"n"+i}}, { group: "edges", data: { id: "e"+i, source: "n0", target: "n"+i } } ); } cy.add(newnodes); any ideas? have used loaded function reload graph not work. not know if have use specific function reload graph. thank in advance. you need add positions new nodes. cytoscape.js docs : it important note positions of newly added nodes must defined when calling cy.add(). nodes can not placed in graph without valid position — otherwise not displayed. yo...
Read more