Fix lookup in SPTree

[alkino]

No description provided.

[azure-pipelines]

✔️ 4e9742e -> Azure artifacts URL

[github-actions]

NEURON ModelDB CI: launching for 4e9742e via its drop url

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 66.31%. Comparing base (ed09605) to head (bbeef5a).

Files	Patch %	Lines
src/nrncvode/sptree.hpp	0.00%	3 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #2759   +/-   ##
=======================================
  Coverage   66.30%   66.31%           
=======================================
  Files         558      558           
  Lines      103970   103970           
=======================================
+ Hits        68942    68945    +3     
+ Misses      35028    35025    -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[nrnhines]

Is this change covered in tests? I don't see a coverage report.

[bbpbuildbot]

Logfiles from GitLab pipeline #195457 (:white_check_mark:) have been uploaded here!

Status and direct links:

• ✅ mac_m1_cmake_build: [cmake, ON, OFF, OFF, address]
• ✅ spack_setup
• ✅ build:nmodl
• ✅ build:neuron:nmodl:intel:legacy
• ✅ build:neuron:nmodl:intel:shared
• ✅ build:neuron:nmodl:nvhpc:acc:legacy
• ✅ build:neuron:nmodl:nvhpc:acc:shared
• ✅ build:neuron:nmodl:nvhpc:omp:legacy
• ✅ build:neuron:nmodl:nvhpc:omp
• ✅ test:neuron:nmodl:intel:legacy
• ✅ test:neuron:nmodl:nvhpc:acc:legacy
• ✅ test:neuron:nmodl:nvhpc:acc:shared
• ✅ test:neuron:nmodl:nvhpc:omp:legacy
• ✅ test:neuron:nmodl:nvhpc:omp
• ✅ test:neuron:nmodl:intel:shared

[alkino]

Is this change covered in tests? I don't see a coverage report.

No, it seems not, I can add one, but it seems to break the modeldb-ci

[nrnhines]

I don't understand how you got External CIs / nrn-modeldb-ci / nrn-modeldb-ci (pull_request) Failing after 126m to become part of the CI.

[nrnhines]

alkino added the nrn-modeldb-ci-nightly label 2 hours ago

I was unaware of that. Very useful!

[sonarcloud]

Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[azure-pipelines]

✔️ bbeef5a -> Azure artifacts URL

[alkino]

In fact the failing models are the same that the one in master...