-
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
Atomese interpreter not needed!? #101
Comments
It should work as follows: and also, |
I agree, @linas, but it was added here due to the experimental nature of that work. |
OK @ngeiswei thanks for looking at the pull reqs. Let me describe what I'm trying to do: ... I want to change the primary data form to that described in the README here: https://github.com/opencog/asmoses/tree/use-values/opencog/asmoses (this is an unmerged branch) An example of reading in a CSV table is here: https://github.com/opencog/asmoses/blob/use-values/opencog/asmoses/data/representation/load_csv.h (please take a look) I'm planing on moving the After that, I'm not sure; just need to follow my nose... The long-term goal is to do MOSES-things at the guile or python prompt, directly (instead of invoking moses binaries). (e.g. issue #89) So I guess to split out the deme mangement and the knob-turning code to make it "generic". (my actual goal is to deploy moses-style mutation on audio & video pipelines) |
Also @Habush fyi. |
This pull req opencog/atomspace#2989 loads CSV/TSV tables into the atomspace. The demo here: https://github.com/opencog/atomspace/blob/master/examples/atomspace/table.scm shows how to use the native atomspace interpreter on these tables. Basically, the "combo" is almost identical to what as-moses uses, except there is an additional |
During a code review, I discovered this code:
https://github.com/opencog/asmoses/blob/master/opencog/asmoses/atomese/interpreter/Interpreter.cc
This interpreter appears to miss the whole idea of atomese: it already has a built-in interpreter, which is called "atomese" -- you just run the code, you don't need an interpreter to run it.
Anyway, this file can be (and should be) removed, because it is impossible for the interpreter to ever know what atom types are actually available in atomese. -- it might have to deal with e.g. DSP types.
The text was updated successfully, but these errors were encountered: