Maniphest T93765

Lineart occlusion doesn't work
Closed, Resolved

Assigned To
YimingWu (NicksBest)
Authored By
Marco Ferrara (marzof)
Dec 6 2021, 9:08 PM
Tags
  • BF Blender
Subscribers
Germano Cavalcante (mano-wii)
Marco Ferrara (marzof)
Pratik Borhade (PratikPB2123)
YimingWu (NicksBest)

Description

System Information
Operating system: Linux-5.11.0-41-generic-x86_64-with-glibc2.31 64 Bits
Graphics card: GeForce GTX 1060 3GB/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 460.91.03

Blender Version
Broken: version: 3.1.0 Alpha, branch: temp-lineart-contained, commit date: 2021-12-01 09:04, hash: rB9afc5ccb6b38
Worked: version: 3.0.0 Release Candidate, branch: master, commit date: 2021-12-02 15:35, hash: 7e60d8a71349

Short description of error
Hi,
I'm testing temp-lineart-contained branch (I'm interested in use_ortho_tolerance feature). It seems that occlusion hiding doesn't work as it should.
As you can see from the attached file the edges on the back of the solid are traced as if they were viewed by the ortho camera.

Strangely enough: if I change the camera type - from ortho to perspective - a rear line is drawn even though it should be hidden by frontal faces.

On master (version: 3.0.0 Release Candidate, branch: master, commit date: 2021-12-02 15:35, hash: 7e60d8a71349) lineart works correctly

Exact steps for others to reproduce the error

  • No steps: just open the attached file and see the lineart on the solid

Related Objects

Mentioned Here
rB7e60d8a71349: Fix missing Blender logo in Windows store package

Event Timeline

Marco Ferrara (marzof) created this task.Dec 6 2021, 9:08 PM
Germano Cavalcante (mano-wii) added subscribers: YimingWu (NicksBest), Germano Cavalcante (mano-wii).Dec 13 2021, 3:37 PM

@YimingWu (NicksBest), is this something of interest to you?

YimingWu (NicksBest) added a comment.EditedDec 15 2021, 5:20 AM

Hi let me check! It seems I broke a lot more than I thought it should. Weird master worked but line art branch doesn't... I shall fix it.

YimingWu (NicksBest) changed the task status from Needs Triage to Confirmed.Dec 15 2021, 6:03 AM
YimingWu (NicksBest) claimed this task.

I pushed a fix for temp-lineart-contained branch, you need to still enable ortho tolerance for everything to work correctly.

A later algorithm rewrite will solve all the problems related here, but for the moment please use this option. :)

Marco Ferrara (marzof) added a comment.Dec 15 2021, 8:58 AM

Thanks @YimingWu (NicksBest), now it seems to be working properly

Pratik Borhade (PratikPB2123) closed this task as Resolved.Mar 22 2022, 8:05 AM
Pratik Borhade (PratikPB2123) added a subscriber: Pratik Borhade (PratikPB2123).
In T93765#1272852, @Marco Ferrara (marzof) wrote:

Thanks @YimingWu (NicksBest), now it seems to be working properly