Skip to content

Support shared library builds of CoreNEURON with GPU support#1922

Merged
pramodk merged 13 commits into
masterfrom
olupton/coreneuron-gpu-dynamic-loading
Aug 28, 2022
Merged

Support shared library builds of CoreNEURON with GPU support#1922
pramodk merged 13 commits into
masterfrom
olupton/coreneuron-gpu-dynamic-loading

Conversation

@olupton

@olupton olupton commented Jul 18, 2022

Copy link
Copy Markdown
Collaborator

@codecov-commenter

codecov-commenter commented Jul 18, 2022

Copy link
Copy Markdown

Codecov Report

Merging #1922 (b437e5b) into master (1b76917) will decrease coverage by 0.00%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##           master    #1922      +/-   ##
==========================================
- Coverage   46.50%   46.50%   -0.01%     
==========================================
  Files         526      526              
  Lines      119216   119231      +15     
==========================================
+ Hits        55439    55445       +6     
- Misses      63777    63786       +9     
Impacted Files Coverage Δ
share/lib/python/neuron/__init__.py 40.23% <40.00%> (-0.01%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@olupton olupton force-pushed the olupton/coreneuron-gpu-dynamic-loading branch 2 times, most recently from 6857061 to 11e8c69 Compare July 26, 2022 10:37
@olupton olupton force-pushed the olupton/coreneuron-gpu-dynamic-loading branch from c9034e0 to 72aada2 Compare August 2, 2022 13:41
@olupton olupton force-pushed the olupton/coreneuron-gpu-dynamic-loading branch from 5185448 to 709ec88 Compare August 19, 2022 14:24
@olupton olupton requested a review from pramodk August 19, 2022 16:53
@olupton olupton marked this pull request as ready for review August 19, 2022 16:54
@olupton olupton force-pushed the olupton/coreneuron-gpu-dynamic-loading branch from 12d57fb to 185839f Compare August 22, 2022 11:45
@azure-pipelines

Copy link
Copy Markdown

✔️ b8392a376864f1c3518e804d658013b3b9b60da5 -> Azure artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ 7588e96d2db0d6a4a8e26b5480683831838c1bcf -> Azure artifacts URL

@olupton olupton changed the title Export compile-time features to Python and add CoreNEURON GPU workarounds Support shared library builds of CoreNEURON with GPU support Aug 23, 2022
@olupton olupton force-pushed the olupton/coreneuron-gpu-dynamic-loading branch from 7588e96 to 73fa3aa Compare August 24, 2022 15:24
@azure-pipelines

Copy link
Copy Markdown

✔️ 73fa3aa0ce09899ea74552a938a77e0fd6d680d5 -> Azure artifacts URL

@olupton olupton force-pushed the olupton/coreneuron-gpu-dynamic-loading branch from 73fa3aa to 1ae331a Compare August 25, 2022 12:46
@azure-pipelines

Copy link
Copy Markdown

✔️ 1ae331a -> Azure artifacts URL

@pramodk pramodk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!
And, generated gpu wheel now works from pure python execution!

Once downstream CI passes and submodules are updated then we are ready to go!

Comment thread share/lib/python/neuron/__init__.py
@azure-pipelines

Copy link
Copy Markdown

✔️ b437e5b -> Azure artifacts URL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants