Open danyaljj opened 8 years ago
These were related to contextual features, specifically when we assumed the sequential relationships between node instances. I think we used a couple of them in the ER example. I believe these should be rewritten and some of them can be removed, but it would need someone who works on a new round of editing the query language. Just removing these without completely working out the related issues in not a good idea, I think.
Just removing these without completely working out the related issues in not a good idea, I think.
Agreed.
I believe these should be rewritten and some of them can be removed, but it would need someone who works on a new round of editing the query language.
Could you give examples of what you have in mind?
Ok, let me overview these all and let you know. thanks!
Small reminder.
Another reminder.
It seems I had missed the first reminder. In general, I think we need to look at the design of the data model from scratch and modify it. This is related to the feature extraction draft paper too. I think we better to keep this issue open for now until we have someone who is interested in working on the datamodel and simple and flexible feature extraction.
@kordjamshidi there are many small details related to "windows" and "ordering the instances".
PropertyWithWindow
: I don't completely understand the purpose of this. Any ideas?Learnable
Node
.contextualFeatureTest
.What should be done with these? An alternative is do get rid of these, and add equivalent methods somewhere (for example inside
DataMode
) and use edges in order to simulate the same behavior.Related to #42 #349