-
Notifications
You must be signed in to change notification settings - Fork 60
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
SequenceLearner does not fulfill API for data #223
Comments
I would recommend holding the indices in a different datastructure if it is really necessary. |
What about these changes? |
I don't think that that really addresses the issue. If |
Like we discussed yesterday, it is not possible to make a Also, |
The keys in
data
for theSequenceLearner
are not just the points in the domain: they are tuples that also contain some index into the sequence of values.This does not conform to the API for
data
which is:where
X
was previously defined to be the domain of the function being learned.The text was updated successfully, but these errors were encountered: