Skip to content
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

Enable trivial cases of global state variables. #1130

Merged
merged 2 commits into from
Jan 15, 2024
Merged

Conversation

1uc
Copy link
Collaborator

@1uc 1uc commented Dec 20, 2023

Adds the functionality required to make trivial cases involving GLOBAL variables work.

@1uc
Copy link
Collaborator Author

1uc commented Dec 20, 2023

This PR includes preparatory commits to move some printing code. Which I'd like to split away once accepted.

It also includes the generated .cpp file as a trial run to see if it helps with understanding what's happening.

@1uc 1uc force-pushed the 1uc/trivial-global-vars branch 2 times, most recently from 9eba6e0 to 59efbf0 Compare December 20, 2023 09:08
@iomaganaris iomaganaris added the NEURON codegen Work toward NEURON code generation label Dec 20, 2023
@1uc 1uc force-pushed the 1uc/trivial-global-vars branch from 84d38e0 to 3cb7a94 Compare December 20, 2023 09:44
Copy link

codecov bot commented Dec 20, 2023

Codecov Report

Attention: 19 lines in your changes are missing coverage. Please review.

Comparison is base (b01e5f7) 88.31% compared to head (6d14759) 88.21%.

Files Patch % Lines
src/codegen/codegen_neuron_cpp_visitor.cpp 56.81% 19 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1130      +/-   ##
==========================================
- Coverage   88.31%   88.21%   -0.10%     
==========================================
  Files         175      175              
  Lines       12981    13024      +43     
==========================================
+ Hits        11464    11489      +25     
- Misses       1517     1535      +18     

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

@bbpbuildbot

This comment has been minimized.

@1uc 1uc marked this pull request as ready for review December 20, 2023 13:55
@1uc 1uc force-pushed the 1uc/trivial-global-vars branch from 3cb7a94 to f118e0d Compare January 9, 2024 15:58
@bbpbuildbot

This comment has been minimized.

@bbpbuildbot

This comment has been minimized.

@1uc 1uc mentioned this pull request Jan 11, 2024
@1uc 1uc force-pushed the 1uc/trivial-global-vars branch from 3255394 to a6799f9 Compare January 11, 2024 09:33
@bbpbuildbot

This comment has been minimized.

@1uc 1uc force-pushed the 1uc/trivial-global-vars branch from a6799f9 to 6d14759 Compare January 12, 2024 09:26
@bbpbuildbot
Copy link
Collaborator

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

Status and direct links:

@1uc 1uc merged commit ef8fcbf into master Jan 15, 2024
20 checks passed
@1uc 1uc deleted the 1uc/trivial-global-vars branch January 15, 2024 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NEURON codegen Work toward NEURON code generation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants