From d08f08924fa5131618aa13efcea16f1b00a3e65a Mon Sep 17 00:00:00 2001 From: Weiqun Zhang Date: Wed, 15 Oct 2025 20:58:14 -0700 Subject: [PATCH] Update ParmParse --- dependencies.json | 2 +- src/Base/ParmParse.cpp | 48 +++++++++++++++++++++--------------------- 2 files changed, 25 insertions(+), 25 deletions(-) diff --git a/dependencies.json b/dependencies.json index 6b79735c..b774b10f 100644 --- a/dependencies.json +++ b/dependencies.json @@ -2,6 +2,6 @@ "version_pyamrex": "25.10", "version_amrex": "25.10", "version_pybind11": "v3.0.1", - "commit_amrex": "26054b26bb653e594835a5a686893f6c20e7c5cd", + "commit_amrex": "42853742514953472f32b762e33eae3c31b1d68d", "commit_pybind11": "v3.0.1" } diff --git a/src/Base/ParmParse.cpp b/src/Base/ParmParse.cpp index d731dfc1..97002785 100644 --- a/src/Base/ParmParse.cpp +++ b/src/Base/ParmParse.cpp @@ -35,30 +35,30 @@ void init_ParmParse(py::module &m) .def_static("addfile", &ParmParse::addfile) - .def("add", py::overload_cast(&ParmParse::add)) - .def("add", py::overload_cast(&ParmParse::add)) - .def("add", py::overload_cast(&ParmParse::add)) - .def("add", py::overload_cast(&ParmParse::add)) - .def("add", py::overload_cast(&ParmParse::add)) - .def("add", py::overload_cast(&ParmParse::add)) - .def("add", py::overload_cast(&ParmParse::add)) - .def("add", py::overload_cast(&ParmParse::add)) - .def("add", py::overload_cast(&ParmParse::add)) - - .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) - .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) - .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) - .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) - .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) - .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) - .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) - .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) + .def("add", py::overload_cast(&ParmParse::add)) + .def("add", py::overload_cast(&ParmParse::add)) + .def("add", py::overload_cast(&ParmParse::add)) + .def("add", py::overload_cast(&ParmParse::add)) + .def("add", py::overload_cast(&ParmParse::add)) + .def("add", py::overload_cast(&ParmParse::add)) + .def("add", py::overload_cast(&ParmParse::add)) + .def("add", py::overload_cast(&ParmParse::add)) + .def("add", py::overload_cast(&ParmParse::add)) + + .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) + .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) + .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) + .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) + .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) + .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) + .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) + .def("addarr", py::overload_cast const &>(&ParmParse::addarr)) // TODO: getters and queries .def("get_bool", [](ParmParse &pp, std::string name, int ival) { bool ref; - pp.get(name.c_str(), ref, ival); + pp.get(name, ref, ival); return ref; }, "parses input values", py::arg("name"), py::arg("ival")=0 @@ -67,7 +67,7 @@ void init_ParmParse(py::module &m) .def("get_int", [](ParmParse &pp, std::string name, int ival) { int ref; - pp.get(name.c_str(), ref, ival); + pp.get(name, ref, ival); return ref; }, "parses input values", py::arg("name"), py::arg("ival")=0 @@ -76,7 +76,7 @@ void init_ParmParse(py::module &m) .def("get_real", [](ParmParse &pp, std::string name, int ival) { amrex::Real ref; - pp.get(name.c_str(), ref, ival); + pp.get(name, ref, ival); return ref; }, "parses input values", py::arg("name"), py::arg("ival")=0 @@ -85,7 +85,7 @@ void init_ParmParse(py::module &m) .def("query_int", [](ParmParse &pp, std::string name, int ival) { int ref; - bool exist = pp.query(name.c_str(), ref, ival); + bool exist = pp.query(name, ref, ival); return std::make_tuple(exist,ref); }, "queries input values", py::arg("name"), py::arg("ival")=0 @@ -155,7 +155,7 @@ void init_ParmParse(py::module &m) [&](auto&& arg) { using T = std::remove_pointer_t>; T v; - pp.get(name.c_str(), v); + pp.get(name, v); add_nested(v, name); }, entry.m_typehint @@ -166,7 +166,7 @@ void init_ParmParse(py::module &m) using T = std::remove_pointer_t>; if constexpr (!std::is_same_v) { std::vector valarr; - pp.getarr(name.c_str(), valarr); + pp.getarr(name, valarr); add_nested(valarr, name); } },