| |
| Anonymous | Login | 2021-01-16 06:36 MSK |  |
| My View | View Issues | Change Log | Roadmap |
| View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||
| 0032042 | Open CASCADE | [OCCT] OCCT:Visualization | public | 2021-01-07 11:20 | 2021-01-12 12:17 | ||||||||
| Reporter | kgv | ||||||||||||
| Assigned To | bugmaster | ||||||||||||
| Priority | normal | Severity | integration request | ||||||||||
| Status | reviewed | Resolution | open | ||||||||||
| Platform | OS | OS Version | |||||||||||
| Product Version | |||||||||||||
| Target Version | [OCCT] 7.6.0* | Fixed in Version | |||||||||||
| Summary | 0032042: Visualization, TKOpenGl - pre-multiply headlight flag into light source position | ||||||||||||
| Description | Currently headlight flag is passed to GLSL program so that transformation has to be applied dynamically. This looks like unnecessary overcomplication - as camera position remains static within frame, it makes sense premultiplying transformation and passing final headlight position. | ||||||||||||
| Tags | No tags attached. | ||||||||||||
| Test case number | |||||||||||||
| Attached Files | |||||||||||||
 Relationships |
||||||
|
||||||
 Relationships |
|
Notes |
|
|
(0098022) git (administrator) 2021-01-09 23:34 |
Branch CR32042 has been created by kgv. SHA-1: c262aafaff5398237d85c2ab434141501460ab70 Detailed log of new commits: Author: kgv Date: Fri Jan 8 00:48:29 2021 +0300 0032042: Visualization, TKOpenGl - pre-multiply headlight flag into light source position OpenGl_ShaderManager::pushLightSourceState() now pre-multiplies headlight transformation. Built-in shading GLSL programs have been updated to remove special handling if headlight flag; also removed redundant pre-normalization of light source direction (already normalized implicitly by Graphic3d_CLight interface). Graphic3d_CLight::SetHeadlight() now raises exception in case of ambient light type. OpenGl_ShaderManager::SetLastView() - removed unused property. |
|
(0098032) git (administrator) 2021-01-10 02:29 |
Branch CR32042 has been updated forcibly by kgv. SHA-1: f9be7ae44ff09900e41e2b41627ab91bdf803fe9 |
|
(0098036) kgv (developer) 2021-01-10 04:25 |
Patch is ready for review - OCCT branch: CR32042. http://jenkins-test-12.nnov.opencascade.com/view/CR32042-master-KGV [^] |
|
(0098088) osa (developer) 2021-01-12 12:17 |
The patch was reviewed |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2021-01-07 11:20 | kgv | New Issue | |
| 2021-01-07 11:20 | kgv | Assigned To | => kgv |
| 2021-01-07 11:23 | kgv | Relationship added | related to 0027750 |
| 2021-01-09 23:34 | git | Note Added: 0098022 | |
| 2021-01-10 02:29 | git | Note Added: 0098032 | |
| 2021-01-10 04:25 | kgv | Note Added: 0098036 | |
| 2021-01-10 04:25 | kgv | Assigned To | kgv => osa |
| 2021-01-10 04:25 | kgv | Status | new => resolved |
| 2021-01-12 12:17 | osa | Note Added: 0098088 | |
| 2021-01-12 12:17 | osa | Assigned To | osa => bugmaster |
| 2021-01-12 12:17 | osa | Status | resolved => reviewed |
|
Issue History |
| Copyright © 2000 - 2021 MantisBT Team |
 |