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

Conversation

@iomaganaris
Copy link
Contributor

Description

Adding documentation related to Hines Matrix solver on GPU and cell permutation

@codecov-commenter
Copy link

Codecov Report

Merging #726 (745a6a0) into master (7b0b6d9) will not change coverage.
The diff coverage is 100.00%.

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

@@           Coverage Diff           @@
##           master     #726   +/-   ##
=======================================
  Coverage   55.51%   55.51%           
=======================================
  Files         107      107           
  Lines        8947     8947           
=======================================
  Hits         4967     4967           
  Misses       3980     3980           
Impacted Files Coverage Δ
coreneuron/permute/cellorder1.cpp 94.25% <ø> (ø)
coreneuron/permute/cellorder2.cpp 94.37% <ø> (ø)
coreneuron/permute/cellorder.cpp 100.00% <100.00%> (ø)

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 7b0b6d9...db924cc. Read the comment docs.

Copy link
Contributor

@kotsaloscv kotsaloscv left a comment

Choose a reason for hiding this comment

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

LGTM

@pramodk pramodk merged commit c868909 into master Dec 23, 2021
@pramodk pramodk deleted the hackathon/magkanar/hines_doc branch December 23, 2021 05:18
pramodk pushed a commit to neuronsimulator/nrn that referenced this pull request Nov 2, 2022
* Initial comments added for the cell permutation algorithms
* Additional comments on permutation strategies

Co-authored-by: Christos Kotsalos <christos.kotsalos@epfl.ch>

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

6 participants