Skip to content
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

Cortical rule modifies vocab #731

Closed
jgosmann opened this issue Jun 10, 2015 · 3 comments
Closed

Cortical rule modifies vocab #731

jgosmann opened this issue Jun 10, 2015 · 3 comments

Comments

@jgosmann
Copy link
Collaborator

When I add a cortical rule like a = b in a SPA module all semantic pointers in a's vocabulary are added to b's vocabulary. That doesn't make sense to me (the other direction would make somewhat sense) and is annoying because I have semantic pointers in a's vocabulary that should not be in b's vocabulary (i.e., the 'YES' pointer of the comparison).

@xchoo
Copy link
Member

xchoo commented Jun 10, 2015

Yup! This issue was brought up in #696, and an attempt to get around this is sort of implemented here, using a read-only flag on vocabularies. #699. (Although, I just noticed that if you try to add to a read only vocabulary, it will raise an error when I think it should raise a warning instead).

@jgosmann
Copy link
Collaborator Author

As this is a duplicate of #696 which has been resolved, this can be closed as well. :)

@tbekolay
Copy link
Member

Feels good seeing all these issues closed! 😍 🎉 🌈

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants