-
Notifications
You must be signed in to change notification settings - Fork 14
Closed
Description
From here:
input_scitype = MMI.Table(MMI.Missing, MMI.Known), # also ok: MMI.Table(Union{MMI.Missing, MMI.Known}),What is written in the comment is correct. What is actually used is not:
julia> X = (; x=[missing, 1, 2])
(x = Union{Missing, Int64}[missing, 1, 2],)
julia> scitype(X) <: Table(Missing, Known)
false
julia> scitype(X) <: Table(Union{Missing, Known})
trueFor more on the Table scitype constructor, see here.
All the tree scitypes need changing.
Sorry that I did not pick this up in my review.
DilumAluthge and sylvaticus
Metadata
Metadata
Assignees
Labels
No labels