Le lun. 8 juil. 2019 à 06:56, John Cowan <xxxxxx@ccil.org> a écrit :
1) Since SRFI 167 is an interface already, I think that the nstore-engine procedure just needs an okvs object and a prefix.

I was very happy with my invention :D It was meant to allow to easily switch okvs during for instance development or tests.
Passing only okvs object and a prefix will make it much more difficult to switch backend.

 
2) In the nstore procedure, clarify that the prefix of the nstore is appended to the prefix of the engine.

There is mis-understanding the prefix in nstore-engine is okvs-prefix-range, I will rename it.
 

3) Change nstore-ask? to nstore-contains?.

Ok



4) I don't see any need for nstore-vars.  Just use symbols.

If one store symbols in the nstore they should be able to query them. It is useful to differentiate symbols from variables.
 
5) It's not clear to me how the variables map onto the non-prefix part of the key/value pair in the underlying okvs.  I think there are two plausible approaches: the first element of each tuple is the key, or the first n elements (where n is specific to the nstore) are the key.  I prefer the second approach.

It depends of the pattern => That is part of the trick! I will try to document that better today.
 
6)  Have nstore-from's generator return lists, eliminating the dependency on SRFI 146.  The car of the list is the key, the cdr is a list of values.

I disagree. It will force to think about the order in which variables appear in nstore-select and nstore-select-where.
 
7)  Change the name of nstore-from to nstore-select, and nstore-select to nstore-select-where.

Ok
 

8) Tuple matching needs to be defined more clearly.  One approach is to use a predicate for each element of the tuple that defines whether this element matches.  This matters for both nstore-from and nstore-where.

Ok
 


John Cowan          http://vrici.lojban.org/~cowan        xxxxxx@ccil.org
"The serene chaos that is Courage, and the phenomenon of Unopened
Consciousness have been known to the Great World eons longer than Extaboulism."
"Why is that?" the woman inquired.
"Because I just made that word up", the Master said wisely.
        --Kehlog Albran, The Profit






--
Amirouche ~ amz3 ~ https://hyper.dev