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

Refactor neuronTypeToString() from Global.cpp #791

Open
stiber opened this issue Feb 5, 2025 · 0 comments
Open

Refactor neuronTypeToString() from Global.cpp #791

stiber opened this issue Feb 5, 2025 · 0 comments
Assignees
Labels
refactor doesn't change functionality, just improves code

Comments

@stiber
Copy link
Contributor

stiber commented Feb 5, 2025

  1. Rename to vertexTypeToString()
  2. Move it to AllVertices
  3. Should this be a utility function or a method?

// MODEL INDEPENDENT FUNCTION NMV-BEGIN {
string neuronTypeToString(vertexType t)
{
switch (t) {
case INH:
return "INH";
case EXC:
return "EXC";
default:
cerr << "ERROR->neuronTypeToString() failed, unknown type: " << t << endl;
assert(false);
return nullptr; // Must return a value -- this will probably cascade to another failure
}
}
// } NMV-END

@stiber stiber added the refactor doesn't change functionality, just improves code label Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor doesn't change functionality, just improves code
Projects
None yet
Development

No branches or pull requests

2 participants