Function is also fixed.

Dilawar Singh · Dilawar Singh · commit 226d61633c75 · 2020-04-03T12:35:32.000+05:30
diff --git a/pybind11/Finfo.cpp b/pybind11/Finfo.cpp
@@ -285,9 +285,20 @@ unsigned int __Finfo__::getNumField()
     return __Finfo__::getNumFieldStatic(oid_, f_);
 }
 
+bool __Finfo__::setNumField(unsigned int num)
+{
+    return __Finfo__::setNumFieldStatic(oid_, f_, num);
+}
+
 unsigned int __Finfo__::getNumFieldStatic(const ObjId& oid, const Finfo* f)
 {
     auto o = ObjId(oid.path() + '/' + f->name());
     return Field<unsigned int>::get(o, "numField");
 }
 
+bool __Finfo__::setNumFieldStatic(const ObjId& oid, const Finfo* f,
+                                  unsigned int num)
+{
+    auto o = ObjId(oid.path() + '/' + f->name());
+    return Field<unsigned int>::set(o, "numField", num);
+}
diff --git a/pybind11/Finfo.h b/pybind11/Finfo.h
@@ -58,8 +58,11 @@ class __Finfo__ {
 
     // Get attribute (python api);
     unsigned int getNumField();
+    bool setNumField(unsigned int);
 
     static unsigned int getNumFieldStatic(const ObjId& oid, const Finfo* f);
+    static bool setNumFieldStatic(const ObjId& oid, const Finfo* f,
+                                  unsigned int i);
 
     // Exposed to python as __setitem__
     bool setItem(const py::object& key, const py::object& val);
diff --git a/pybind11/pymoose.cpp b/pybind11/pymoose.cpp
@@ -149,7 +149,7 @@ PYBIND11_MODULE(_cmoose, m)
     py::class_<__Finfo__>(m, "_Finfo", py::dynamic_attr())
         .def(py::init<const ObjId &, const Finfo *, const char *>())
         .def_property_readonly("type", &__Finfo__::type)    
-        .def_property_readonly("num", &__Finfo__::getNumField)   // Only for FieldElementFinfos
+        .def_property("num", &__Finfo__::getNumField, &__Finfo__::setNumField)   // Only for FieldElementFinfos
         .def("__call__", &__Finfo__::operator())
         .def("__call__", &__Finfo__::operator())
         .def("__getitem__", &__Finfo__::getItem)
diff --git a/tests/py_moose/test_function.py b/tests/py_moose/test_function.py
@@ -29,6 +29,7 @@ def test_var_order():
     dt = 1.0
     # fn0 = moose.Function('/fn0')
     fn1 = moose.Function('/fn1')
+    print(fn1.x.num)
     fn1.x.num = 2
     fn1.expr = 'y1+y0+x1+x0'
     fn1.mode = 1

Original file line number	Diff line number	Diff line change
`@@ -285,9 +285,20 @@ unsigned int __Finfo__::getNumField()`
`285`	`285`	`return __Finfo__::getNumFieldStatic(oid_, f_);`
`286`	`286`	`}`
`287`	`287`
	`288`	`+bool __Finfo__::setNumField(unsigned int num)`
	`289`	`+{`
	`290`	`+ return __Finfo__::setNumFieldStatic(oid_, f_, num);`
	`291`	`+}`
	`292`	`+`
`288`	`293`	`unsigned int __Finfo__::getNumFieldStatic(const ObjId& oid, const Finfo* f)`
`289`	`294`	`{`
`290`	`295`	`auto o = ObjId(oid.path() + '/' + f->name());`
`291`	`296`	`return Field<unsigned int>::get(o, "numField");`
`292`	`297`	`}`
`293`	`298`
	`299`	`+bool __Finfo__::setNumFieldStatic(const ObjId& oid, const Finfo* f,`
	`300`	`+ unsigned int num)`
	`301`	`+{`
	`302`	`+ auto o = ObjId(oid.path() + '/' + f->name());`
	`303`	`+ return Field<unsigned int>::set(o, "numField", num);`
	`304`	`+}`