-
Notifications
You must be signed in to change notification settings - Fork 33
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
Feat/nsgr integration #243
Conversation
381389a
to
772bf19
Compare
So users can just change the argument of this method and not have to modify the method itself in their scripts/code
For the moment, I used the HH example from docs to test it out. It was submitted and ran fine but threw up this error that I now need to debug:
Can reproduce on my local machine too, will debug and see what's causing it. TODO for this PR: add a test |
A simple test in the pyneuroml/examples directory:
fails because the output is expected in a |
It seems these are not re-generated by jnml if they already exist, and so, they're not picked up here. Simplest to delete them all so that they are always regenerated
Just makes it easier to test
TODO: tweak netpyne run |
This function - gets a list of all the LEMS/NeuroML files that are required - copies them to a new, clean directory - generates all the simulation engine related files - creates a zip with all these to send to NSG - sends to NSG If it is not interrupted, when the job finishes, the data from the job is also downloaded to the new runner directory.
This is now ready for review. It doesn't rely on newly generated files. It creates a new directory and does all its work there, so it should not be affected by any old files at all. To test netpyne, in the
Note that netpyne on nsgr is too old from the looks of it: NeuroML/NeuroML2#213 |
Copy commands do not re-create missing directories in the destination path, so we need to re-create them manually.
No description provided.