Skip to content
This repository was archived by the owner on May 9, 2024. It is now read-only.

Spinnaker start #52

Merged
merged 25 commits into from
May 7, 2024
Merged

Spinnaker start #52

merged 25 commits into from
May 7, 2024

Conversation

rowleya
Copy link
Member

@rowleya rowleya commented Mar 7, 2024

Time to get this into master?

@rowleya rowleya requested a review from Christian-B March 7, 2024 15:35
@Christian-B
Copy link
Member

Any reason not to add these to Integrations tests?

Specificaly script_builder.py

Copy link
Member

@Christian-B Christian-B left a comment

Choose a reason for hiding this comment

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

test!

@rowleya
Copy link
Member Author

rowleya commented Mar 8, 2024

All of these applications open a GUI. It would be good at some point to separate the operation from the GUI, but that hasn't been done so far. Issue SpiNNakerManchester/PyNNExamples#115 created to at least keep track of this.

@Christian-B
Copy link
Member

pendulum_follow_c_vis.py raises ValueError
On application vertex source, the number of dimensions in the maximum number of atoms per core (9223372036854775807,) must be the same number of dimensions as that of the vertex shape (120, 120)

@Christian-B
Copy link
Member

suduko
spinn_utilities.exceptions.ConfigException: Unexpected None for section='Database' option='wait_on_confirmation_timeout'

@rowleya
Copy link
Member Author

rowleya commented Mar 15, 2024

These should be working now...

@Christian-B Christian-B merged commit c565653 into master May 7, 2024
6 checks passed
@Christian-B Christian-B deleted the spinnaker_start branch May 7, 2024 06:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants