Skip to content
This repository was archived by the owner on Mar 20, 2023. It is now read-only.

Update nmodl submodule: fixes for WATCH statements on GPU#598

Merged
pramodk merged 4 commits into
masterfrom
pramodk/watch-codegen-fixes
Jul 23, 2021
Merged

Update nmodl submodule: fixes for WATCH statements on GPU#598
pramodk merged 4 commits into
masterfrom
pramodk/watch-codegen-fixes

Conversation

@pramodk

@pramodk pramodk commented Jul 21, 2021

Copy link
Copy Markdown
Collaborator

See BlueBrain/nmodl/pull/710 and BlueBrain/nmodl/issues/679

Todos:

How to test this?

Run testwatch.hoc test from testcorenrn repository

Test System

  • OS: BB5
  • Compiler: default nvhpc + CUDA
  • Backend: GPU

Use certain branches for the SimulationStack CI

CI_BRANCHES:NEURON_BRANCH=pramodk/watch-codegen-fixes,NMODL_BRANCH=pramodk/watch-codegen-fixes

@pramodk pramodk requested review from iomaganaris and olupton July 21, 2021 21:17
@olupton

olupton commented Jul 23, 2021

Copy link
Copy Markdown
Contributor

GitLab integration seems a bit confused. The pipeline was duplicated, I killed one but https://bbpgitlab.epfl.ch/hpc/coreneuron/-/pipelines/11640 is still running -- not sure if the status here will go green or not 🤷

@pramodk pramodk merged commit 96a22c9 into master Jul 23, 2021
@pramodk pramodk deleted the pramodk/watch-codegen-fixes branch July 23, 2021 16:34
@pramodk

pramodk commented Jul 23, 2021

Copy link
Copy Markdown
Collaborator Author

The last failure is because I didn't wait for one more CI just because of submodule update in NMODL. It was already validated.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants