Nanostring-Biostats / InSituType

An R package for performing cell typing in SMI and other single cell data
Other
29 stars 11 forks source link

bug: what should be NAs in logliks are called 0 #119

Closed patrickjdanaher closed 2 years ago

patrickjdanaher commented 2 years ago

In these lines of insitutype:

out$logliks <- matrix(0, nrow(counts), length(uniqueclusts),
                      dimnames = list(rownames(counts), uniqueclusts))

The "0" should be NA instead.

When running the multi-tissue mode, this gives 0 values to clusters that aren't even under consideration for the tissue.

After this update, the flightpath functions should be tested on multi-tissue outputs to make sure they're working.