".." in cognostic name breaks Trelliscope

[jrounds]

This can probably be filed under "Well don't do that?" But it is something that cost me like 2 hours of my life, so I thought I would mention it =). If you are wondering how I did that it was a gsub to clean-up a colname and I accidentally left an extra dot in the name.

This is the recreatable for me. You view and try to look at it and on my computer nothing happens. Cognostics arn't sortable, some interactions work, some don't. etc.

data(iris)
div = divide(iris,by="Species")
cogFn = function(v){
ret = list()
ret[["name01.."]] = cog(1)
return(ret)
}
panelFn = function(v){
plot(1:10)
}
conn = vdbConn("vdb", name = "_", autoYes=TRUE)
makeDisplay(data = div,
name = "Breaking_Telliscope",
cogFn = cogFn,
panelFn = panelFn,
desc = "Breaking Trelliscope With Odd Names",
conn = conn
)
view()

[hafen]

Thanks for pointing this out and sorry about those 2 hours! Landon Sego added some checking along these lines for display names a while back. We should add a check, warning, or conversion for cognostics as well. Would you mind filing an issue on github to this effect?

Thanks,

Ryan

On Aug 25, 2015, at 12:44 AM, jrounds [email protected] wrote:

This can probably be filed under "Well don't do that?" But it is something that cost me like 2 hours of my life, so I thought I would mention it =). If you are wondering how I did that it was a gsub to clean-up a colname and I accidentally left an extra dot in the name.

This is the recreatable for me. You view and try to look at it and on my computer nothing happens. Cognostics arn't sortable, some interactions work, some don't. etc.

data(iris)
div = divide(iris,by="Species")
cogFn = function(v){
ret = list()
ret[["name01.."]] = cog(1)
return(ret)
}
panelFn = function(v){
plot(1:10)
}
conn = vdbConn("vdb", name = "_", autoYes=TRUE)
makeDisplay(data = div,
name = "Breaking_Telliscope",
cogFn = cogFn,
panelFn = panelFn,
desc = "Breaking Trelliscope With Odd Names",
conn = conn
)
view()

—
Reply to this email directly or view it on GitHub #118.