Skip to content
Snippets Groups Projects
Commit 8566655c authored by Ben Cumming's avatar Ben Cumming Committed by Sam Yates
Browse files

Fix GPU event delivery bugs (#394)

* Properly iterate over all event queues in the case where a cell group has more than once cell (triggered in e.g. GPU miniapp).
* Update GPU unit test to use the new `cell_group::advance()` interface.
parent 87a52776
No related branches found
No related tags found
No related merge requests found
...@@ -115,7 +115,7 @@ time_type model::run(time_type tfinal, time_type dt) { ...@@ -115,7 +115,7 @@ time_type model::run(time_type tfinal, time_type dt) {
PL(); PL();
PE("enqueue"); PE("enqueue");
threading::parallel_for::apply(0, num_groups(), threading::parallel_for::apply(0, communicator_.num_local_cells(),
[&](cell_size_type i) { [&](cell_size_type i) {
const auto epid = epoch_.id; const auto epid = epoch_.id;
merge_events( merge_events(
......
...@@ -26,7 +26,7 @@ TEST(mc_cell_group, test) ...@@ -26,7 +26,7 @@ TEST(mc_cell_group, test)
{ {
mc_cell_group<fvm_cell> group({0u}, cable1d_recipe(make_cell())); mc_cell_group<fvm_cell> group({0u}, cable1d_recipe(make_cell()));
group.advance(epoch(0, 50), 0.01); group.advance(epoch(0, 50), 0.01, {});
// the model is expected to generate 4 spikes as a result of the // the model is expected to generate 4 spikes as a result of the
// fixed stimulus over 50 ms // fixed stimulus over 50 ms
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment