search

frogg / STUPRO

Satellite-based terrain visualization for environmental monitoring and presentation of relevant observations
Other
6 stars 3 forks source link

Feature/geodaeten refactoring #161

Closed qw3ry closed 8 years ago

qw3ry commented 8 years ago

Bitte drueberschauen. Insbesondere Naming und Umstaendlichkeit vom Code pruefen - ich hab einiges hin und herjonglieren muessen und bin mir nicht sicher ob das so gut ist.

Ergaenzung: Wegen dem default-Wert in globe.radius sind die Boegen in der Flachkarte kaum zu sehen. Im Globe-Modus passt aber alles, der Rest wird von @Marukyu gerade gefixt

Ausserdem fehlt noch die Fluglaenge als Attribut, die sich der Kunde am Montag gewuenscht hat.

qw3ry commented 8 years ago

Bump.

dieterpl commented 8 years ago

Hey Tobi, ich habe deinen Code angeschaut und getestet ob alles funktioniert. Dabei ist mir aufgefallen das bei einem LOD von kleiner als 0.5 zum Teil Geodäten durch die Kugel gezeichnet werden. Außerdem scheint der sphericalToCartesian Filter bei ausgestellter transform Funktion nicht die daten auf Karten größe zu reduzieren. (möglicherweiße schon gefixed) Dadurch konnte ich die Flachkarte nicht wirklich testen. Ansonsten funktioniert alles recht gut auch dein code ist gut strukturiert und kommentiert.

qw3ry commented 8 years ago

Okay. Ich hab grad den develop nochmal reingemerged. Das mit dem S2C-Filter ist hiermit gefixt, flachkarte sollte also jetzt gehen.

Dass die Geodäten durch die Kugel gehen liegt daran, dass bei niedrigem LoD nur eine (bzw. wenige) gerade Linie(n) von Start- zu Endflughafen gezeichnet werden. Die gehen dann natürlich durch die Erde durch (wenn die Flughäfen weit genug auseinander sind). IMHO ist das Teil des Konzepts.

qw3ry commented 8 years ago

Damit werden jetzt auch die Arrays weitergereicht. Ich bin mir nicht ganz sicher ob das so wirklich korrekt funktioniert, aber ich fürchte wir müssen jetzt damit leben.

qw3ry commented 8 years ago

Hab die Arrays doch noch gefixt. getSize() != getNumberOfTuples() :(