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

GPU build improvements for NEURON integration #373

Merged
merged 5 commits into from
Aug 17, 2020

Conversation

pramodk
Copy link
Collaborator

@pramodk pramodk commented Aug 15, 2020

  • Improvemenets for GPU build integration with NEURON
    • create property CORENEURON_LIB_LINK_FLAGS with necessary LDFLAGS
      to link coreneuron with neuron via nrnivmodl
    • suppress additional PGI warnings which are not necessary to emit
      with old mod2c
    • minor change in nrn_setup.hpp : functions declarated as static
      in the header result into warning as definitions are in .cpp file

  - MPI C++ bindings are removed from MPI
  - avoid linking mpi c++ library for simplicity
  - print mpi c/c++ compiler
  - create property CORENEURON_LIB_LINK_FLAGS with necessary LDFLAGS
    to link coreneuron with neuron via nrnivmodl
  - suppress additional PGI warnings which are not necessary to emit
    with old mod2c
  - minor change in nrn_setup.hpp : functions declarated as static
    in the header result into warning as definitions are in .cpp file
@pramodk pramodk force-pushed the pramodk/gpu-build-improvements branch 2 times, most recently from 8586b59 to 2214a82 Compare August 16, 2020 08:11
@pramodk pramodk force-pushed the pramodk/gpu-build-improvements branch from 2214a82 to cb5ed7f Compare August 16, 2020 08:12
@pramodk pramodk marked this pull request as ready for review August 16, 2020 08:15
@pramodk
Copy link
Collaborator Author

pramodk commented Aug 16, 2020

Also include #372, should be merged after that.

Copy link
Contributor

@ohm314 ohm314 left a comment

Choose a reason for hiding this comment

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

looks all good!

@ohm314 ohm314 merged commit dca1dc0 into master Aug 17, 2020
@alkino alkino deleted the pramodk/gpu-build-improvements branch August 17, 2020 22:00
pramodk added a commit to neuronsimulator/nrn that referenced this pull request Nov 2, 2022
* Update README with latest workflow

* Avoid linking to C++ MPI libraries
  - MPI C++ bindings are removed from MPI
  - avoid linking mpi c++ library for simplicity
  - print mpi c/c++ compiler

* Improvemenets for GPU build integration with NEURON
  - create property CORENEURON_LIB_LINK_FLAGS with necessary LDFLAGS
    to link coreneuron with neuron via nrnivmodl
  - suppress additional PGI warnings which are not necessary to emit
    with old mod2c
  - minor change in nrn_setup.hpp : functions declarated as static
    in the header result into warning as definitions are in .cpp file

* Update weights from GPU before copying to NEURON

CoreNEURON Repo SHA: BlueBrain/CoreNeuron@dca1dc0
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants