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.

Sign up for GitHub

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

Jump to bottom

Add float32 array EclSum.to_numpy #820

Closed
wants to merge 2 commits into from
Closed

Add float32 array EclSum.to_numpy #820

wants to merge 2 commits into from

Commits on Aug 11, 2021

  1. Add float32 array EclSum.to_numpy 

    This commit introduces pybind11 in order to accomplish the goal of
dumping float32 arrays. This could've been done by adding yet another C
function accessible from Python via cwrap. However, I made the decision
to have the Python API not depend on the C API directly, instead adding
Python bindings via pybind11. This allows us to extend the Python API
without exposing implementation details in C and avoids having the
Python API do manual memory management to deal with C.
    @pinkwah
    pinkwah committed Aug 11, 2021
    Configuration menu
    Copy the full SHA
    bcf1491 View commit details
    Browse the repository at this point in the history

  2. : numpy_vector don't cast to double

    @pinkwah
    pinkwah committed Aug 11, 2021
    Configuration menu
    Copy the full SHA
    bde4892 View commit details
    Browse the repository at this point in the history