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

Mergetempwrefrac #13

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Mergetempwrefrac #13

wants to merge 7 commits into from

Conversation

ionefine
Copy link
Contributor

Hi Geoff, More more progress today. There's a couple of things you could usefully work on.

  1. If you run Beauchamp_Cell2020 there is a bug. I've flagged it so it will just show up if you run it. It seems that the hack I made in generate pulse train to get rid of the sneaky little zeros between the anodic and cathodic does something nasty occasionally.
  2. I've emailed you a copy of the paper. There are two places where I have a comment in there, which is where we describe the temporal model. One should be as brief as possible, the other should be more detailed.
  3. Finally, tomorrow we should talk about what should go in the final bit of the paper. I think we should definitely put the Troyk in there, since our predictions are going to be right. But we could also characterize some useful stuff. One issue is small electrodes. Our model suggests that there's a certain size, and going below that is pointless. I think we should calculate that. Another is the spacing between the electrodes (related to the Troyk paper). Basically the distance on the cortical surface that is needed to have the phosphene move by 1/2 it's standard deviation (or something like that). That makes the point that really tiny spacing between electrodes is also pointless. Finally I guess we could take the 'optimal model' and simulate a movie. None of these would be difficult to do at this point. Let's chat tomorrow?

@gboynton gboynton closed this Feb 25, 2023
@gboynton gboynton reopened this Feb 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants