-
Notifications
You must be signed in to change notification settings - Fork 66
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
Allow changing data type of EQLs Jacobian elements #176
Labels
enhancement
Idea or request for a new feature
Comments
6 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description of the desired feature
We know that fitting an EQL gridder is a process that requires a fair amount of memory, mainly to store the Jacobian matrix.
One small step into reducing the required memory would be to change the data type used to store the Jacobian elements, which are defined as
float64
by default.Currently, the EQLs offer no option to change that, even though the
jacobian
method has an optional argumentdtype
to change it.Would be nice to have a
jac_dtype
argument that could be passed to the EQL constructor in order to change the representation of the Jacobian elements.Although the memory consumption won't drop by a lot, users may reduce it by half storing the elements as
float32
.I suspect that using
float32
won't significantly compromise the accuracy of the predictions.Are you willing to help implement and maintain this feature? Yes
The text was updated successfully, but these errors were encountered: