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

Wrong render result of 'Dimension pre-rendered block' #122

Closed
Sululu666 opened this issue Nov 22, 2024 · 3 comments
Closed

Wrong render result of 'Dimension pre-rendered block' #122

Sululu666 opened this issue Nov 22, 2024 · 3 comments

Comments

@Sululu666
Copy link

The problem is when open this file, some dimension part not display.
zhongwangpeixun1.zip

Here is the result:
1.by other product:
image

2.by dxf-viewer:
image

It took a while found that when not use dimension pre-rendered block,
image

those part will show:
image
(with wrong text as actualMeasurement of dimension not handle)

Question:
Is there any lack of judgement to decide whether use 'Dimension pre-rendered block', like the content integrity of block?
If it is because development is not complete, please ignore this issue.

@vagran
Copy link
Owner

vagran commented Nov 22, 2024

Thanks for the report and sample file. I do not think the problem is in the pre-rendering block selection, most probably something in the block is not rendered properly for some reason (maybe not implemented). Rendering dimensions in fall-back path (without the block) is quite tricky and in most cases produces a result which is not identical to one in the CAD software (dimension rendering usually is very software-specific, that is probably the reason these blocks are introduced in DXF and used in most files). So will need to investigate why the block is not rendered correctly.

@Sululu666
Copy link
Author

Thanks for the answer.
By the way, I've found that those pre-renderd blocks' color and colorIndex are both undefined
and for temporary test I set the color to 'byLayer.
企业微信截图_17323338052003

then the render result is right:
image

hope this give you train of thought

@vagran
Copy link
Owner

vagran commented Nov 28, 2024

Seems to be fixed by #123. You can check it on demo page which has the recent version deployed.

@vagran vagran closed this as completed Nov 28, 2024
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

2 participants