Python: impactx.__version__ (#494)

* Test: `impactx.__version__`

Make sure this is not empty.

* Fix: `impactx.__version__`

Set a default value and fallback.

Author: ax3l

CMakeLists.txt

@@ -80,7 +80,7 @@ option(ImpactX_amrex_internal                    "Download & build AMReX" ON)
 option(ImpactX_openpmd_internal                  "Download & build openPMD-api" ON)
 option(ImpactX_pyamrex_internal                  "Download & build pyAMReX" ON)
 
-set(pyImpactX_VERSION_INFO "" CACHE STRING
+set(pyImpactX_VERSION_INFO "${ImpactX_VERSION}" CACHE STRING
     "PEP-440 conformant version (set by setup.py)")
 
 # change the default build type to Release (or RelWithDebInfo) instead of Debug

src/python/pyImpactX.cpp

@@ -8,6 +8,10 @@
 #include <ImpactX.H>
 #include <particles/elements/All.H>
 
+#ifndef PYIMPACTX_VERSION_INFO
+#   include <ImpactXVersion.H>
+#endif
+
 #define STRINGIFY(x) #x
 #define MACRO_STRINGIFY(x) STRINGIFY(x)
 
@@ -25,7 +29,7 @@ void init_transformation(py::module&);
 
 PYBIND11_MODULE(impactx_pybind, m) {
     // make sure AMReX types are known
-    py::module::import("amrex.space3d");
+    auto amr = py::module::import("amrex.space3d");
 
     m.doc() = R"pbdoc(
             impactx_pybind
@@ -47,12 +51,16 @@ PYBIND11_MODULE(impactx_pybind, m) {
     init_transformation(m);
     init_ImpactX(m);
 
+    // expose our amrex module
+    m.attr("amr") = amr;
+
     // API runtime version
     //   note PEP-440 syntax: x.y.zaN but x.y.z.devN
 #ifdef PYIMPACTX_VERSION_INFO
     m.attr("__version__") = MACRO_STRINGIFY(PYIMPACTX_VERSION_INFO);
 #else
-    m.attr("__version__") = "dev";
+    // note: not necessarily PEP-440 compliant
+    m.attr("__version__") = IMPACTX_VERSION;
 #endif
 
     // authors

tests/python/test_impactx.py

@@ -12,9 +12,18 @@
 import pytest
 
 import amrex.space3d as amr
+import impactx
 from impactx import ImpactX, RefPart, distribution, elements
 
 
+def test_impactx_module():
+    """
+    Tests the basic modules we provide.
+    """
+    print(f"version={impactx.__version__}")
+    assert impactx.__version__  # version must not be empty
+
+
 def test_impactx_fodo_file():
     """
     This tests an equivalent to main.cpp in C++