Joannes Lemmens

Klopt, in ICT-5159 ga ik alle DND's implementeren.

Klopt, in ICT-5159 ga ik alle DND's implementeren.

[ICT-5157] 3D-MVX: Opvullijsten
[ICT-5157] 3D-MVX: Opvullijsten
Dit is wat outdated: de nieuwe waarde is "34 - 7". Op deze manier ligt het schroefgat van de geleider 7mm boven het center van de opvullijst, waar ook net het gat van de opvullijst zich bevindt. Id...

Dit is wat outdated: de nieuwe waarde is "34 - 7". Op deze manier ligt het schroefgat van de geleider 7mm boven het center van de opvullijst, waar ook net het gat van de opvullijst zich bevindt.
Idem voor de rechtse opvullijst.

De reden waarom ik die commit hier niet heb toegevoegd is omdat er anders weer een hoop extra meekwam (reviewopmerkingen van vorige tickets).

Dit is kwestie van voorkeur, maar voor zoiets zou ik een variabele aanmaken 'ShowGalleryFront' die je hier dan gebruikt en/of de laatste twee condities combineren naar: oneOf(string(BoxSystemHeig...

Dit is kwestie van voorkeur, maar voor zoiets zou ik een variabele aanmaken 'ShowGalleryFront' die je hier dan gebruikt en/of de laatste twee condities combineren naar:

oneOf(string(BoxSystemHeight), 'C', 'D'))


Het is zeker niet nodig want dit is vlot leesbaar.

Heel goede opmerking. Ik weet niet meer goed van waar het kwam. Waarschijnlijk inderdaad op zicht bepaald, maar ik had het nu overgenomen van de front reling. Ik heb het aangepast, ook voor de fron...

Heel goede opmerking. Ik weet niet meer goed van waar het kwam. Waarschijnlijk inderdaad op zicht bepaald, maar ik had het nu overgenomen van de front reling.
Ik heb het aangepast, ook voor de front reling.

[ICT-5156] 3D-MVX: dwarsreling
[ICT-5156] 3D-MVX: dwarsreling
string() is nodig voor numerieke waarden zoals BoxSystemDepth, maar niet voor waarden die al strings zijn zoals CutOutCode_Z1. Ik heb het even nagekeken, en het lijkt dat ik dit consistent doe voor...

string() is nodig voor numerieke waarden zoals BoxSystemDepth, maar niet voor waarden die al strings zijn zoals CutOutCode_Z1.
Ik heb het even nagekeken, en het lijkt dat ik dit consistent doe voor enkel numerieke waarden.

[ICT-5151] 3D-MVX: Sifonkappen
[ICT-5151] 3D-MVX: Sifonkappen
Dit lijkt me in sectie 'MATERIAL / DO NOT DELIVER' te moeten staan ipv bij de ProductId's

Dit lijkt me in sectie 'MATERIAL / DO NOT DELIVER' te moeten staan ipv bij de ProductId's

[ICT-5149] 3D-MVX: Stalen rug
[ICT-5149] 3D-MVX: Stalen rug
Ik heb dit kenmerk moeten toevoegen aan 'NeededFor3D' omdat voor een of andere reden de andere niet altijd ingevuld was.

Ik heb dit kenmerk moeten toevoegen aan 'NeededFor3D' omdat voor een of andere reden de andere niet altijd ingevuld was.

De Z-as in Encoway is de 'diepte-as', spiegelen wilt zeggen dat de voorkant de achterkant wordt en omgekeerd. Dit is omdat de Z-as uit het beeld komt, waardoor alle Z-translaties negatief zijn. De ...

De Z-as in Encoway is de 'diepte-as', spiegelen wilt zeggen dat de voorkant de achterkant wordt en omgekeerd.
Dit is omdat de Z-as uit het beeld komt, waardoor alle Z-translaties negatief zijn. De lengte van dit onderdeel moet echter wel positief zijn (ik heb een negatieve lengte uitgeprobeerd, maar dit zag er niet uit).
Hierdoor begint de contour op het nulpunt, maar wordt dan langs 'de foute kant' uitgetrokken (naar voor toe), waarbij twee opties zijn om ze naar achter te krijgen:
1) Een translatie over de Z-as gelijk aan de lengte
2) Een spiegeling t.o.v. de Z-as

Ik heb de tweede, en toegegeven minst voor de hand liggende, optie gekozen.

Edit: ik heb bij het verwerken van deze opmerkingen de eerste optie gekozen sinds die logischer is.

Dat is zeker een goede suggestie! Op zich vond ik mijn formules nog wel mooi sinds de start- en eindcoördinaten (x-y)/2 en (x+y)/2 zijn, waarbij (1) hun gemiddelde x/2 is, (2) de afstand tussen de...

Dat is zeker een goede suggestie!

Op zich vond ik mijn formules nog wel mooi sinds de start- en eindcoördinaten (x-y)/2 en (x+y)/2 zijn, waarbij
(1) hun gemiddelde x/2 is,
(2) de afstand tussen de twee is y is.
Waarbij dus x = BaseWidth en y = SynchroLinkageLength.

Maar ik snap dat de intentie erachter niet onmiddellijk duidelijk is.

p.s.: de '-6' erachter komt doordat de pinnetjes die aan de bridge zitten links en rechts niet dezelfde grootte hebben.

[ICT-5155] 3D-MVX: Zijstabilisering
[ICT-5155] 3D-MVX: Zijstabilisering
Goede opmerking, ik heb ze gecombineerd

Goede opmerking, ik heb ze gecombineerd

Ik heb ze toegevoegd op svn en ook aan deze review toegevoegd mocht je ze nog willen bekijken.

Ik heb ze toegevoegd op svn en ook aan deze review toegevoegd mocht je ze nog willen bekijken.

[ICT-5153] 3D-MVX: AfdekkapRugwandHouder
[ICT-5153] 3D-MVX: AfdekkapRugwandHouder
Deze wijziging zit in commit 154 die ik niet mee in deze review heb gestoken, maar omdat die tussen commit 153 en 155 zit komt die blijkbaar toch nog tevoorschijn... Mijn excuses hiervoor.

Deze wijziging zit in commit 154 die ik niet mee in deze review heb gestoken, maar omdat die tussen commit 153 en 155 zit komt die blijkbaar toch nog tevoorschijn... Mijn excuses hiervoor.

[ICT-5150] 3D-MVX: Geleiders
[ICT-5150] 3D-MVX: Geleiders