-
Notifications
You must be signed in to change notification settings - Fork 30
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
Double check protected data member #69
Comments
See this for reference. |
The current library rework is addressing this. |
To date, if you change the scope of unsigned int num_particle_;
std::size_t state_size_;
ParticleSet pred_particle_;
ParticleSet cor_particle_; in @xEnVrE we need to discuss |
Many classes have way too much
protected
data member that should probably be moved inprivate
scope. See for exampleGaussianMixture
and derived classes.cc @xEnVrE
The text was updated successfully, but these errors were encountered: