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.

Sign up for GitHub

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

Jump to bottom

Fixes for new data structures #2

Merged
merged 1 commit into from
Apr 30, 2023
Merged

Fixes for new data structures #2

merged 1 commit into from
Apr 30, 2023

Conversation

olupton
Copy link
Contributor

@olupton olupton commented Dec 12, 2022 •
edited
Loading

See neuronsimulator/nrn#2027 for more context.

@olupton olupton mentioned this pull request Dec 13, 2022
Merged
6 tasks
@olupton olupton mentioned this pull request Feb 27, 2023
Merged
@olupton olupton changed the title CNAME: don't assume strings live long enough Fixes for new data structures Apr 20, 2023
@olupton
Fixes for new data structures
f53e63b 
Also cope with forward declared Prop
@olupton olupton marked this pull request as ready for review April 20, 2023 19:28
@olupton olupton requested review from ramcdougal and RuiLi7222 April 20, 2023 19:57
@ramcdougal ramcdougal merged commit 2f2786a into master Apr 30, 2023
@ramcdougal ramcdougal deleted the olupton/fixes branch April 30, 2023 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ramcdougal ramcdougal Awaiting requested review from ramcdougal

@RuiLi7222 RuiLi7222 Awaiting requested review from RuiLi7222

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@olupton @ramcdougal