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

Application Framework - Invalid document after partial read of XBF #201

Draft
wants to merge 1 commit into
base: IR
Choose a base branch
from

Conversation

dpasukhi
Copy link
Member

@dpasukhi dpasukhi commented Dec 18, 2024

Resolve missed refereces until all are resolved to produce expected
result without exceptions.
XCAFDoc_ShapeTool::GetShape() throws Standard_NullObject exception after
partial read of XBF document

XCAFDoc_ShapeTool::GetShape() throws Standard_NullObject exception when XBF document is read partially using functionality of PCDM_ReaderFilter.

The problem is that XBF driver resolves N+1 level of referenced items only while XCAFDoc_ShapeTool::GetShape() requires to follow the deeper chain of references.

…en-Cascade-SAS#201

Resolve missed refereces until all are resolved to produce expected
  result without exceptions.
XCAFDoc_ShapeTool::GetShape() throws Standard_NullObject exception after
  partial read of XBF document
@dpasukhi dpasukhi added 2. Bug Something isn't working 1. Application Framework CAF/XCAF/OCAF and other cases related with OCCT document labels Dec 18, 2024
@dpasukhi dpasukhi self-assigned this Dec 18, 2024
@dpasukhi
Copy link
Member Author

Needs to refactor code and check careful with "while" scenario.
Additionally needs to move variables from the class field to method body if possible.

@dpasukhi dpasukhi marked this pull request as draft December 18, 2024 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. Application Framework CAF/XCAF/OCAF and other cases related with OCCT document 2. Bug Something isn't working
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

1 participant