search

neuronsimulator / nrn-modeldb-ci

NEURON ModelDB CI tools
BSD 3-Clause "New" or "Revised" License
4 stars 3 forks source link

`runmodels`: add `--norun` option #32

Closed alexsavulescu closed 2 years ago

alexsavulescu commented 2 years ago

Closes #31

Note I have only done some sanity checking (didn't run all models)

alexsavulescu commented 2 years ago

Note I have only done some sanity checking (didn't run all models)

running here: https://github.com/neuronsimulator/nrn-modeldb-ci/actions/runs/2453365351

LE: looks fine

alexsavulescu commented 2 years ago

I'll have to think about the purpose of the "Skipped runs":

I remember some of those were because the simulation was taking too long. So I curated a few and enabled them. The rest didn't have any comments or were not straightforward to shorten, therefore still skipped.

LE: opened #33 for this

alexsavulescu commented 2 years ago

i.e. https://github.com/neuronsimulator/nrn-modeldb-ci/blob/master/modeldb/modeldb-run.yaml#L414

alexsavulescu commented 2 years ago

I have run locally all models with --norun and did not spot any issue. Will merge.