-
Notifications
You must be signed in to change notification settings - Fork 116
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move codebase to CPP #763
Move codebase to CPP #763
Conversation
67d9bc3
to
f99a341
Compare
71ada5b
to
ef7513f
Compare
313a09f
to
44c78f2
Compare
For reasons that elude me at this point in time, 5 rxd tests are failing for the autotools jobs:
more here: https://travis-ci.com/github/neuronsimulator/nrn/jobs/414446958#L8937 |
How does one run one of the autotools tests. For cmake, |
For autotools I use (on mac) : cd nrn
git checkout cpp
./build.sh
./configure --without-x --with-paranrn=dynamic --with-nrnpython=`which python` --with-readline=no --prefix=`pwd`/install
make
make install
export PYTHONPATH=`pwd`/install/lib/python:$PYTHONPATH
export PATH=`pwd`/install/x86_64/bin:$PATH
python share/lib/python/neuron/rxdtests/run_all.py
I don't think those tests are run with cmake; by running |
@nrnhines : Just to be sync : @adamjhn just informed us following:
So, if simple hh results are different between autotools and CMake then I suspect we have missed something fundamental. We haven't checked details yet. (Note that this different happens only this pull request branch; master works correctly) |
I will block some dedicated hours to go though all those 458 files 👀. All changes are straightforward but just to be sure we don't miss anything.
And then we should be ready to go 🚀 |
I'll also
|
@alexsavulescu : Having bit curated commit message would be helpful for this mega merge? May be you can take a look at the log and copy message here? @nrnhines @alexsavulescu : another question - should we merge all (or most of the) commits from current master to release/8.0 before this merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still WIP
This sums it up:
Sure, save ourselves some cherry-picking |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Half way through...!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few minor comments/questions otherwise LGTM
@pramodk so I'm happy if you're happy. CI providers have been erratic these past days (GHA doubling jobs, failed actions for no reason, Azure reporting failed jobs but they weren't...) but everything is a-OK. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pramodk so I'm happy if you're happy.
😃 ✅ 🚀
More details in #708.