electron-rare/FreeCAD
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Materials: Set array quantity format (#21647)

Browse Source 
* Materials: Set array quantity format

When setting quantity formats, the default format is not actually the
default. Instead fixed format is, which causes issues for very small
and very large values. This was fixed for C++ but arrays created in
Python were not setting the format correctly.

* Apply suggestions from code review

Co-authored-by: Benjamin Nauck <[email protected]>
This commit is contained in:
David Carter
2025-06-04 16:27:37 +00:00
committed by GitHub
co-authored by Benjamin Nauck
parent 7105a2332e
commit 657da35e6b
2 changed files with 21 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/Mod/Material/App/Array2DPyImp.cpp
+3 -1
View File
@@ -159,7 +159,9 @@ PyObject* Array2DPy::setValue(PyObject* args)
if (PyArg_ParseTuple(args, "iiO!", &row, &column, &PyUnicode_Type, &valueObj)) {
Py::String item(valueObj);
try {
QVariant variant = QVariant::fromValue(Base::Quantity::parse(item.as_string()));
auto quantity = Base::Quantity::parse(item.as_string());
quantity.setFormat(MaterialValue::getQuantityFormat());
QVariant variant = QVariant::fromValue(quantity);
getArray2DPtr()->setValue(row, column, variant);
}
catch (const InvalidIndex&) {
src/Mod/Material/App/Array3DPyImp.cpp
+18 -6
View File
@@ -163,10 +163,16 @@ PyObject* Array3DPy::setDepthValue(PyObject* args)
if (PyArg_ParseTuple(args, "iO!", &depth, &PyUnicode_Type, &valueObj)) {
Py::String item(valueObj);
try {
getArray3DPtr()->setDepthValue(depth, Base::Quantity::parse(item.as_string()));
auto quantity = Base::Quantity::parse(item.as_string());
quantity.setFormat(MaterialValue::getQuantityFormat());
getArray3DPtr()->setDepthValue(depth, quantity);
}
catch (const InvalidIndex&) {
PyErr_SetString(PyExc_IndexError, "Invalid array index");
catch (const Base::ParserError& e) {
PyErr_SetString(PyExc_ValueError, e.what());
return nullptr;
}
catch (const InvalidIndex& e) {
PyErr_SetString(PyExc_IndexError, e.what());
return nullptr;
}
Py_Return;
@@ -185,10 +191,16 @@ PyObject* Array3DPy::setValue(PyObject* args)
if (PyArg_ParseTuple(args, "iiiO!", &depth, &row, &column, &PyUnicode_Type, &valueObj)) {
Py::String item(valueObj);
try {
getArray3DPtr()->setValue(depth, row, column, Base::Quantity::parse(item.as_string()));
auto quantity = Base::Quantity::parse(item.as_string());
quantity.setFormat(MaterialValue::getQuantityFormat());
getArray3DPtr()->setValue(depth, row, column, quantity);
}
catch (const InvalidIndex&) {
PyErr_SetString(PyExc_IndexError, "Invalid array index");
catch (const Base::ParserError& e) {
PyErr_SetString(PyExc_ValueError, e.what());
return nullptr;
}
catch (const InvalidIndex& e) {
PyErr_SetString(PyExc_IndexError, e.what());
return nullptr;
}
Py_Return;