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.

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

PMI annotations reading (TDF_Label.FindAttribute from OCP not working properly) #153

Open
GabrielJMS opened this issue Aug 14, 2024 · 2 comments

Comments

@GabrielJMS
Copy link

Hello,

I'm trying to implement the reading of PMI annotations (Geometrical Dimensions and tolerances) from a step file using OCP.

I have a C++ code that is working fine and that I have already tested:
annotations

In my C++ code, I have something like this:

m_gdtTool = XCAFDoc_DocumentTool::DimTolTool(doc->Main());

TDF_LabelSequence labels_datums, label_geoms, label_dims, label_dimTols;
m_gdtTool->GetDatumLabels(labels_datums);
m_gdtTool->GetDimensionLabels(label_dims);
m_gdtTool->GetGeomToleranceLabels(label_geoms);

int datumSize = labels_datums.Length();
int dimSize = label_dims.Length();
int geomSize = label_geoms.Length();
// Dimensional Tolerances
for (int i = 1; i <= dimSize; ++i)
{
    const TDF_Label& label_dim = label_dims.Value(i);
    Handle(XCAFDoc_Dimension) aDimAttr;
    label_dim.FindAttribute(XCAFDoc_Dimension::GetID(), aDimAttr);

    if (!aDimAttr.IsNull())
    {
        Handle(XCAFDimTolObjects_DimensionObject) aDimObject = aDimAttr->GetObject();
    }
}

So, I have translated into python:

        labels_datums = TDF_LabelSequence()
        labels_dims = TDF_LabelSequence()
        labels_geoms = TDF_LabelSequence()

        self.gdt_tool.GetDatumLabels(labels_datums)
        self.gdt_tool.GetDimensionLabels(labels_dims)
        self.gdt_tool.GetGeomToleranceLabels(labels_geoms)

        datum_size = labels_datums.Length()
        dim_size = labels_dims.Length()
        geom_size = labels_geoms.Length()
         for i in range(1, dim_size + 1):
            label_dim = labels_dims.Value(i)
            aDimAttr = XCAFDoc_Dimension()
            found = label_dim.FindAttribute(XCAFDoc_Dimension.GetID_s(), aDimAttr)

            if found:
                aDimObject = aDimAttr.GetObject()

When the execution gets in the line aDimObject = aDimAttr.GetObject() I get this error:

image

I have verified and compared the label_dim variable in both python and c++ :

python

image

c++

image

Do you have any ideas why I'm getting this error?

I suspect that the problem is coming from TDF_Label.FindAttribute method:

I think that for my case it's falling in the third case, and maybe the pybind11 wrapping of this method is not working properly because the generic type?
image

I'm attaching the c++ project and the step file used to test the reading. The main file is called TestOCCT.cpp

Thank you very much for the attention. Any insight on how to solve this issue or workaround will be highly appreciated!
TestOCCPMI.zip

@adam-urbanczyk adam-urbanczyk transferred this issue from CadQuery/cadquery Aug 14, 2024
@GabrielJMS GabrielJMS changed the title PMI annotations implementation (TDF_Label.FindAttribute from OCP not working properly) PMI annotations reading (TDF_Label.FindAttribute from OCP not working properly) Aug 14, 2024
@GabrielJMS
Copy link
Author

Maybe the problem is in the line 59:
image

@GabrielJMS
Copy link
Author

GabrielJMS commented Aug 14, 2024

The workaround was to reimplement the FindAttribute method in python using OCP

    def find_attribute(label, guid):
        """
        Reimplementation of the TDF_Label::FindAttribute method in Python.

        :param label: The label to search in.
        :type label: TDF_Label
        :param guid: The GUID of the attribute to find.
        :type guid: Standard_GUID

        :return: A tuple where the first element is a boolean
            indicating if the attribute was found, and the second element is the found attribute handle.
        :rtype: (bool, TDF_Attribute)
        """
        if label.IsNull():
            raise Standard_NullObject("A null Label has no attribute.")

        itr = TDF_AttributeIterator(label)

        while itr.More():
            attr = itr.Value()
            if attr.ID().IsSame(guid):
                return True, attr
            itr.Next()

        return False, None

Does anyone have a suggestion for refactoring the C++ FindAttribute method to improve its compatibility with pybind11? Please let me know if you do, so I can propose a change to the OpenCASCADE project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant