-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Get rid of arma::fill::zeros
when we upgrade the minimum armadillo version
#3681
Comments
Agreed, the only thing is we need to look and see when we can use |
I recommend upgrading the minimum version of Armadillo to 10.8 as soon as Ubuntu 24.04 LTS comes out, scheduled for end of this month. This would cover the two most recent Ubuntu LTS releases.
|
Agreed, at that time it is also worth checking through the codebase for any ancient Armadillo compatibility code (sometimes it gets missed). |
@conradsnicta we are scheduled to move to C++17 this summer, probably in June of July, at that time, we can take care of this as well. (Specifically with the end of RHEL 7) |
This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions! 👍 |
Hi,
This issue is only a placeholder for the above task, that we will probably do when we move to C++17 sometime this summer.
The text was updated successfully, but these errors were encountered: