You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
First, it wouldn't hurt to understand whether Ollie's fix in ContactNetwork.__convert_array_to_dict solved the correct issue. I'm just not sure it wouldn't affect something else that relies on nodes being consecutive integers, for example.
Second, the error that Jinlong pointed out seems to stem from something else (index out of bounds in line 155 in populations.py). At least it's not clear to me why the fix in ContactNetwork.__convert_array_to_dict would fix it. I believe, to fix Jinlong's error it would be enough to write in lines 29-30 of _master_eqn_init.py:
I'm basically making notes to myself to clear it up later; we've had enough of "ah, it works, whatever" moments that turn into "oh, this is clearly not the way it was supposed to be!" moments, so I wanna make sure everything is as expected.
The text was updated successfully, but these errors were encountered:
dburov190
changed the title
Investigate further sorting of the nodes in user base
Investigate further sorting of the nodes in user base case
Oct 14, 2020
First, it wouldn't hurt to understand whether Ollie's fix in
ContactNetwork.__convert_array_to_dict
solved the correct issue. I'm just not sure it wouldn't affect something else that relies on nodes being consecutive integers, for example.Second, the error that Jinlong pointed out seems to stem from something else (index out of bounds in line 155 in
populations.py
). At least it's not clear to me why the fix inContactNetwork.__convert_array_to_dict
would fix it. I believe, to fix Jinlong's error it would be enough to write in lines 29-30 of_master_eqn_init.py
:instead of
but I'm not sure.
I'm basically making notes to myself to clear it up later; we've had enough of "ah, it works, whatever" moments that turn into "oh, this is clearly not the way it was supposed to be!" moments, so I wanna make sure everything is as expected.
The text was updated successfully, but these errors were encountered: