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

Add Population class to main #42

Closed
4 tasks done
clorton opened this issue Sep 25, 2024 · 0 comments · Fixed by #66
Closed
4 tasks done

Add Population class to main #42

clorton opened this issue Sep 25, 2024 · 0 comments · Fixed by #66
Assignees

Comments

@clorton
Copy link
Contributor

clorton commented Sep 25, 2024

  • needs a new name since we also use this class for node/patch data
  • remove deprecated add_property() since we have add_scalar_property() and add_vector_property() now
  • make sure add_vector_property() conforms to issue Transpose array allocation in add_vector_property() (in population.py) #38
  • move any functions that depend on property names, e.g. eliminate_eulas() to a utility rather than part of the class
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 a pull request may close this issue.

1 participant