nrnhines

Results 397 comments of nrnhines

I tried out the script on ```olupton/more-c++```. On first try, the script stopped at merging master-before-format with ``` hines@hines-T7500:~/neuron/format/build$ sh ../../format-branch.sh olupton/more-c++ ... Automatic merge failed; fix conflicts and then...

@alexsavulescu @pramodk > release/8.1 will be created from the very last master commit which will have the formatting I notice that was not done or listed in #1719 . Also...

The formatting was accomplished with two PRs #1748 and #1732 ``` commit 4d9a02539d13636c8e27cc123846072837d760be Author: Olli Lupton Date: Wed Mar 30 09:49:13 2022 +0200 clang-format: one-off for extra files. (#1748) *...

Disturbing thing 1 ``` git clone [email protected]:neuronsimulator/nrn temp cd temp mkdir build cd build cmake .. -G Ninja -DCMAKE_INSTALL_PREFIX=install -DPYTHON_EXECUTABLE=`which python3` -DCMAKE_BUILD_TYPE=Debug -DNRN_ENABLE_RX3D=OFF -DNRN_CLANG_FORMAT=ON ninja clang-format ``` generates ``` Could...

> but where the result of the automatic merge was not formatted "correctly". A very interesting point that did not occur to me. Thanks. I'll try to remember to run...

Sorry for the oversight. I'll port the autotools support for music to cmake.

@ohm314 @mdjurfeldt I started out thinking the ideal cmake option would be ```-DNRN_ENABLE_MUSIC=ON``` but I see that the old autotools option was ```--with-music=/prefix/of/music/installation``` . The simplest way to proceed is...

I was toying with an addition to https://nrn.readthedocs.io/en/latest/cmake_doc/options.html#mpi-options of the form ``` NRN_PATH_TO_MUSIC:STRING= ------------------------- Enable MUSIC. MUlti SImulation Coordinator. The full /path to the MUSIC installation must be specified. See...

I started a branch ```hines/enable-music```. The ```nrn/cmake/FindMusic.cmake``` is mostly nonsense at this point as it contains the hardwired prefix for the music installation. Ie. on my machine: ``` set(MUSIC_PREFIX "/home/hines/soft/MUSIC/install")...