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

Heat demo #3

Open
wants to merge 29 commits into
base: master
Choose a base branch
from
Open

Heat demo #3

wants to merge 29 commits into from

Conversation

dkfellows
Copy link
Member

This is the (converted) visualiser for the Heat Demo.

The code isn't the most pythonic ever, but it is in a far far better state than the original C++ (well, mostly C) code that we started out with. Improvements to it (e.g., to use SpiNNFrontEndCommon and the glut_framework.py in this repository) are very welcome indeed.

Copy link
Member

@rowleya rowleya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This falls at the first hurdle - it doesn't work at all so far...

@rowleya
Copy link
Member

rowleya commented Aug 25, 2017

I have attempted to fix some issues, but it seems to crash when packets start coming in

@rowleya
Copy link
Member

rowleya commented Aug 25, 2017

That said, the old visualiser also seems to fail...

@dkfellows dkfellows added this to the 5.1.0 milestone Aug 12, 2019
@Christian-B Christian-B modified the milestones: 5.1.0, 6.0.0 Jul 8, 2020
@dkfellows
Copy link
Member Author

Still needs quite a bit of work; stuff that should be in a single class is spread over several files.

@dkfellows dkfellows marked this pull request as draft March 24, 2021 12:26
This now gets the “Works On My Machine!” stamp of approval
Also, a mechanism for registering methods as simple-case callbacks for
key and mouse events, which gets rid of some large and messy methods by
breaking them up into smaller named pieces.

Which is nice. https://youtu.be/XOhZgAPn_CU
@dkfellows dkfellows marked this pull request as ready for review March 29, 2021 15:21
@dkfellows
Copy link
Member Author

This now runs for me. I don't know if it works with the back-end correctly.

@dkfellows dkfellows requested a review from rowleya March 29, 2021 15:22
@dkfellows dkfellows added this to the 7.0.0 milestone Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants