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

feat(generate-if-curve): allow specifying segment and fraction along #235

Merged
merged 1 commit into from
Jul 5, 2023

Conversation

sanjayankur31
Copy link
Member

To allow analysis of multi-compartmental cells with inputs at different locations

To allow analysis of multi-compartmental cells with inputs at different
locations
@sanjayankur31 sanjayankur31 added T: enhancement Type: enhancement S: ready for review Status: ready for review labels Jul 4, 2023
@pgleeson
Copy link
Member

pgleeson commented Jul 4, 2023

@sanjayankur31 Maybe they should be segid=0 and fract=0.5 by default instead of None?

@sanjayankur31
Copy link
Member Author

We decided to leave it as None since non compartmental cell models would not take these values. The schema docs say that 0 and 0.5 will be used if these fields are left empty. Merging this now.

@sanjayankur31 sanjayankur31 merged commit d0f044a into development Jul 5, 2023
@sanjayankur31 sanjayankur31 deleted the feat/if-curve-allow-specifying-segment branch July 5, 2023 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S: ready for review Status: ready for review T: enhancement Type: enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants