DiscussionsIssue archiveOCCT:Modeling Data

Archived issue #0028144

Modeling Data, GeomAdaptor_Curve - regression in comparison of version OCCT654

Open CASCADEOCCT:Modeling Dataclosed11 public notes

Search issues

Description

The problem is in translation of face №128(entity 507:#0000516). It looks the rage of edge is computed incorrectly (it was OK in OCCT654).

Steps to reproduce

restore [locate_data_file Attachment 3 (BREP)] a
axo; fit
vdisplay a; vfit

Public activity

11 archived notes

Participants are labeled by their role within this record.

01Commenter 1
Face 128 read from entity 507 is valid. But AXON viewer shows this face as invalid. Please see attached image.
This due to that first edge in the face based on the periodic curve has range -1.31441524331422e-011 0.214529066155427.
02Commenter 2
Problem described in issue is reproduced on current state of OCCT.
03Commenter 3
The problem was actually not in STEP translation but in visualization of the edge with specific range. The same face (translated with current master) is shown without edge in OCCT 7.3.0 (both AXO and V3d viewers) but is OK with current master. The issue thus should be considered as fixed.

I pushed branch CR28144 containing corrected test case for this issue: it reads BREP file instead of STEP (this should be both faster and not depending on translation), and uses wireframe in 3D viewer (the problem is not visible in shading). Please add the attached BREP file to the test databsse and consider the branch for integration.
04Commenter 3
Branch [archived branch] has been created by Participant.

[revision removed]


Detailed log of new commits:

Author: abv
Date: Wed Sep 4 22:57:24 2019 +0300

    0028144: Visualization - regression in comparison of version OCCT654
    
    Test case moved to grid vis and corrected to use BREP instead of STEP, and to use wireframe in 3D viewer
05Commenter 1
> The problem was actually not in STEP translation
> but in visualization of the edge with specific range.
Sorry, I'm unable following this logic.
The fixed bug #0030273 was within GeomAdaptor_Curve::Intervals() algorithm, introduced by #0023062.

06Commenter 1
> tests/bugs/vis/bug28144
Test should be moved to a proper group (moddata_X).
07Commenter 7
>> Sorry, I'm unable following this logic.

The logic is straightforward: I did not delve into research of the rootcause of the problem, just noticed that it is not due to STEP and problem manifests in visualization.
08Commenter 3
Branch [archived branch] has been updated forcibly by Participant.

[revision removed]
09Commenter 9
Test renamed to be in the same grid as test for #30273; branch CR28144 rebased on current IR (OCCT-740beta2). Please review again
10Commenter 3
Branch [archived branch] has been updated forcibly by Commenter 1.

[revision removed]
11Commenter 3
Branch [archived branch] has been deleted by Participant.

[revision removed]

Related records