search

quarree100 / qScope_frontend

Code for the projection of Q-Scope/CityScope@QUARREE100
0 stars 0 forks source link

Switch nrows ncols #11

Closed dunland closed 3 years ago

dunland commented 3 years ago

Der Code erzeugt bei mir in kombination mit dem entsprechenden branch in cspy zusammenpassende Ergebnisse. Allerdings bin ich mir unsicher, ob dadurch die Logik der Grid-Erzeugung noch bewahrt wird, oder ob ich sie durcheinander bringe..

--> run_q100viz.py Zeile 73: grid_1 = session.grid_1 = grid.Grid(canvas_size, nrows, ncols, [[50, 0], [50, 81], [100, 81], [100, 0]], viewport)

pls review!

Vergleichsansicht: https://github.com/dunland/q100_viz/commit/91bba7063af22ba9b50c8cb3acc2521bed0e9b5a?branch=91bba7063af22ba9b50c8cb3acc2521bed0e9b5a&diff=split

dunland commented 3 years ago

die Grid-Initialisierung geht doch etwas schief:

grid_1 = session.grid_1 = grid.Grid(canvas_size, nrows, ncols, [[50, 0], [50, 81], [100, 81], [100, 0]], viewport)

→ Screenshot from 2021-08-24 11-42-26

Es stehen genau 4 Spalten über.

Workaround:

grid_1 = session.grid_1 = grid.Grid(canvas_size, nrows, ncols, [[50, 0], [50, 100], [91, 100], [91, 0]], viewport) # workaround

→ Screenshot from 2021-08-24 11-44-38

janbehrens commented 3 years ago

Der Fehler war in read_scanner_data, wo das Grid (eindimensional) in ein 2D-Array gelesen wird. Siehe ff5a1d0

janbehrens commented 3 years ago

Workaround:

Das ist kein Workaround, sondern genau so gedacht. Die Werte in dst_points dienen der Kalbrierung des Grid relativ zur Tischkante.

dunland commented 3 years ago

Workaround:

Das ist kein Workaround, sondern genau so gedacht. Die Werte in dst_points dienen der Kalbrierung des Grid relativ zur Tischkante.

Habe ich grundsätzlich auch so verstanden. In meinem Code-Vorschlag hat allerdings [100,100] dazu geführt, dass das Grid über die ROI bzw die Tischkante hinausragte. Ist aber mit deinem Fix behoben ✓