-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Efficient Table Representation #16
Comments
Implemented in singnet#3 |
If you get the urge to implement BoolValue, that's OK, I guess. Initially, I wanted to stay minimalist. |
Also, it is OK to use the special-purpose key Note that some values are designed to be time-changing... |
Restructure comboreduct
The various representations suggested in issues #3, #12 and #14 are great for
reasoning but not so great for efficient calculations, thus the
following suggestion: Represent column values (i.e. values associated
to each feature) as a list of values living in the atom feature
itself. For instance assume we have table
The values feature
f1
would be represented as the list[0,1,0]
attached to
f1
via theAtom::setValue
method. The key could beand the ProtoAtom value could be
FloatValue
iff1
is numericalLinkValue
iff1
is Boolean, in such caseTrueLink
orFalseLink
could be used to representtrue
andfalse
. Analternative would be to implement
BoolValue
that holds directlyboolean C++ values which would be more efficient.
That representation could be obtained directly from a Table or from
the various existing representation. Since reasoning isn't needed yet
it could be fine to obtain it directly from the Table.
An another thing we'll want to support is to represent duplicated rows
in the same manner that CTable does, but that's for another time and
another issue.
The text was updated successfully, but these errors were encountered: