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

Create tests #4

Open
dloney opened this issue May 22, 2020 · 2 comments
Open

Create tests #4

dloney opened this issue May 22, 2020 · 2 comments
Labels
enhancement New feature or request

Comments

@dloney
Copy link
Collaborator

dloney commented May 22, 2020

No description provided.

@dloney dloney added the enhancement New feature or request label May 22, 2020
@dloney
Copy link
Collaborator Author

dloney commented May 22, 2020

I'm not that familiar with C++ testing libraries. It looks like Boost has a recommended library. If we introduce a Boost dependency, it can be leveraged with #7 since Boost also provides serialization libraries.

@dloney dloney mentioned this issue May 25, 2020
@dloney
Copy link
Collaborator Author

dloney commented May 25, 2020

If we don't use boost, do you have another method you would prefer as a testing library?

Also #7 will require substantial effort to serialize the model objects. I think this is necessary for long term code development to be able to fully synchronize the models between workers. Boost has a component which helps with that. So it would be introducing one dependency to solve two issues. We can potentially set Boost up as a git submodule and incorporate into the makefile, so the user wouldn't have to link to anything from the main package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant