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

Fix NVHPC 21.9 compilation of some models. - #679

Merged
olupton merged 1 commit into
masterfrom
olupton/nvhpc-21.9-compat
Oct 27, 2021
Merged

olupton merged 1 commit into
masterfrom
olupton/nvhpc-21.9-compat

Conversation

@olupton

@olupton olupton commented Oct 25, 2021 •

Copy link
Copy Markdown
Contributor

Description
See the description of neuronsimulator/nrn#1515, which summarises the many issues and PRs...

Use certain branches for the SimulationStack CI

CI_BRANCHES:NEURON_BRANCH=olupton/reduced-dentate,

@olupton
olupton force-pushed the olupton/nvhpc-21.9-compat branch from 0b24210 to d1ded91 Compare October 25, 2021 13:58
@codecov-commenter

codecov-commenter commented Oct 25, 2021 •

Copy link
Copy Markdown

Codecov Report

Merging #679 (1d8fc02) into master (4b95cca) will not change coverage.
The diff coverage is n/a.

❗ Current head 1d8fc02 differs from pull request most recent head d1ded91. Consider uploading reports for the commit d1ded91 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master     #679   +/-   ##
=======================================
  Coverage   55.25%   55.25%           
=======================================
  Files         107      107           
  Lines        8718     8718           
=======================================
  Hits         4817     4817           
  Misses       3901     3901           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4b95cca...d1ded91. Read the comment docs.

@olupton olupton closed this Oct 25, 2021
@olupton olupton reopened this Oct 25, 2021
@olupton

olupton commented Oct 25, 2021

Copy link
Copy Markdown
Contributor Author

Retest this please Jenkins.

@olupton
olupton marked this pull request as ready for review October 26, 2021 16:43
@olupton
olupton force-pushed the olupton/nvhpc-21.9-compat branch from 5ce5e2c to 5741875 Compare October 27, 2021 09:38

@pramodk pramodk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM once submodules are updated

@olupton
olupton force-pushed the olupton/nvhpc-21.9-compat branch from 5741875 to dc508db Compare October 27, 2021 15:23
* Update mod2c submodule with mod2c#73, which does the same thing there.
* Update nmodl submodule with nmodl#762, which fixes code generation for
  the SH_na8st.mod mechanism in the reduced_dentate model.
@olupton
olupton force-pushed the olupton/nvhpc-21.9-compat branch from dc508db to 03a0056 Compare October 27, 2021 15:26
@olupton
olupton merged commit d2f4d3d into master Oct 27, 2021
@olupton
olupton deleted the olupton/nvhpc-21.9-compat branch October 27, 2021 17:37
@olupton olupton mentioned this pull request Oct 28, 2021
5 tasks done
pramodk pushed a commit to neuronsimulator/nrn that referenced this pull request Nov 2, 2022
This fixes compilation with NVHPC 21.9 in some cases.

* Update mod2c submodule with mod2cBlueBrain/CoreNeuron#73, which does the same thing there.
* Update nmodl submodule with nmodlBlueBrain/CoreNeuron#762, which fixes code generation for
  the SH_na8st.mod mechanism in the reduced_dentate model.

CoreNEURON Repo SHA: BlueBrain/CoreNeuron@d2f4d3d
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