search

sapo17 / BachelorThesis

Simple material parameter estimator tool using Mitsuba 3 and Dr.Jit.
2 stars 0 forks source link

Bug: Optimize checkbox causes crash on Linux #56

Closed sapo17 closed 1 year ago

sapo17 commented 1 year ago

Bug

Optimize checkbox (shown below) causes a crash on Linux (tested: WSL Ubuntu). The main problem seems to be adding QWidget into QTable cell (table.setCellWidget(...)).

Although this is not an issue in Windows, the application must work on both OSes.

problem

sapo17 commented 1 year ago

Fix: Changed selection of to-be optimized scene parameters. The new selection scheme requires a double click on the vertical header label, i.e. the scene parameter label on the row (e.g. a double click on red.reflectance.value toggles whether the scene parameter should be optimized in the procedure).

fix

The to-be optimized parameters are highlighted with a gold color. Color scheme chosen according to this article (Coloring for color blindness)

See commit

sapo17 commented 1 year ago

Additional fixes, see commit