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

Wrong semantic pointer plot values when showing pairs #1020

Open
HugoChateauLaurent opened this issue Oct 4, 2019 · 5 comments
Open

Wrong semantic pointer plot values when showing pairs #1020

HugoChateauLaurent opened this issue Oct 4, 2019 · 5 comments

Comments

@HugoChateauLaurent
Copy link

Description

When showing pairs of semantic pointers, the value of activation of the semantic pointers is not correct.

To reproduce

  1. Run the attached file
  2. Toggle "Show pairs" of the SP plot on and off

nengo_bug.zip

Expected behavior

The activation of the pointers should be the same whether pairs are shown or not.

Screenshots

nengo_bug

Versions

  • OS: Ubuntu 16.04
  • Browser: Chrome
  • Nengo GUI: 0.4.4 (from pip)
  • Nengo: 2.8.0 (from pip)
  • Nengo SPA: 0.6.1 (from pip)
@arvoelke
Copy link
Contributor

arvoelke commented Oct 4, 2019

For the expected behaviour, is it fair to say that on the right-hand legend the "B" should still be green (and ~0.97), and the "A*B" should be orange (and ~0.16) instead? That is, it's an issue with the colours (and ~values) in the legend?

@HugoChateauLaurent
Copy link
Author

Yes the legend seems to be the problem here.

@jgosmann
Copy link
Collaborator

jgosmann commented Oct 4, 2019

This is a long known bug (#903), among many others (#768, #737, #709). I'm still hoping to provide SPA plots that work more robustly in Nengo SPA at some point, but this hinges on a rewrite of Nengo GUI which I don't know the state of. Given the current Nengo GUI code base, I at least do not want spend my free time on fixing this.

@HugoChateauLaurent
Copy link
Author

Ok sorry, I wasn't aware of the referenced issues.

@drasmuss
Copy link
Member

drasmuss commented Oct 7, 2019

Hi Hugo, thanks for the report. We're in the midst of a significant rewrite of the Nengo GUI, so bugfixes and things like that will be fixed a bit more slowly than usual until that is complete. We'll take a look and see if this is something that could be fixed by a quick patch, but we're trying not to invest too much time into updating the old code base for now (since most of it will end up being replaced). But we'll definitely make sure that this is working properly in the updated Nengo GUI!

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

No branches or pull requests

4 participants