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

subprocess.Popen need shell=True on windows. #34

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nrnhines
Copy link
Member

From a windows install of NEURON,
bash-5.1$ runmodels --norun --workdir foo 3670
succeeds in the sense of

$ cd foo/NTW_NEW
$ nrniv
NEURON -- VERSION 8.2a-109-gff2af9afd+ olupton/more-c++ (ff2af9afd+) 2022-06-09
Duke, Yale, and the BlueBrain Project -- Copyright 1984-2021
See http://neuron.yale.edu/neuron/credits

loading membrane mechanisms from E:\hines\models\nrn-modeldb-ci\foo\NTW_NEW\nrnmech.dll
Additional mechanisms from files
 HH2.mod IAHP.mod ICAN.mod IT2.mod capump.mod gabaa.mod gabab1.mod
oc>

and in foo.json, one can see that the compilation occurred.

@alexsavulescu
Copy link
Member

alexsavulescu commented Jun 17, 2022

Just launched the whole CI for hines/windows-modelrun here: https://github.com/neuronsimulator/nrn-modeldb-ci/actions/runs/2515415377

If all OK (in the report), will merge

@alexsavulescu alexsavulescu marked this pull request as draft June 17, 2022 13:33
@alexsavulescu
Copy link
Member

The CI got stuck, we should investigate this locally.

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