search

mcdougallab / matlabneuroninterface

Interface for connecting NEURON and MATLAB
BSD 3-Clause "New" or "Revised" License
5 stars 1 forks source link

Small documentation updates #34

Closed AljenU closed 1 year ago

AljenU commented 1 year ago

Looking at the DEV_README.md, it looks like the instructions there assume that the dll has been copied to our "source" directory first. However, this should not be necessary. If, in those instructions, you replace the two instances of "source/libnrniv.dll" with the location of the libnrniv.dll within the NEURON installation folder, I expect the instructions to work just fine.

AljenU commented 1 year ago

When trying to run build_interface on windows, with mingw selected as both c and c++ compiler, I still got an error:

An error occurred while initializing child process: While attempting to execute "REDACTED\matlabneuroninterface\2a03-961f-b749-17a2.exe": CreateProcessW: Access is denied

So, we should add in the documentation, where mentioning (how to) run build_interface, that you need Administrator rights, and / or must have started Matlab as an adminstrator user, in order to be able to build the interface. Since at least clibgen.generateLibraryDefinition() needs that.

edovanveen commented 1 year ago

Fixed in edovanveen-phase2a_docupdates (https://github.com/edovanveen/matlabneuroninterface/tree/phase2a_docupdates).

edovanveen commented 1 year ago

Closed by pull request #42