This is a list of all comments for CR-2895.

Review Summary: No summary


----------------------------------------
File: development/Visualisatie_3D/Parametric_3D_models/MVX.json
Revision Comment by Matthias Claessens on 27 August 2025, 07:36 
http://subversion02:8060/cru/CR-2895#c4507

  Deze mapping wordt niet gebruikt in de visualisatie hieronder.
  Je hebt mogelijks gewoon alle sifon kenmerken gemapt die
  beschikbaar waren.
  Nu blijkt dat deze niet gebruikt worden zou ik deze
  verwijderen.

Revision Comment by Matthias Claessens on 27 August 2025, 07:39 
http://subversion02:8060/cru/CR-2895#c4508

  Dit werkt duidelijk, ik zie de kap in de 3D visualisatie.
  
  Dat wil zeggen dat je niet {{string(CutOutCode_Z1)}} moet doen.
  Nu voor de andere doe je dit wel expliciet.
  
  Het is misschien wat muggenziften maar zou het beter zijn om
  dit allemaal uniform te hebben?
  Dus allemaal zonder string(), of allemaal met string()

  Reply by Joannes Lemmens on 27 August 2025, 12:46
  > 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.

Revision Comment by Matthias Claessens on 27 August 2025, 07:44 
http://subversion02:8060/cru/CR-2895#c4509

  Deze +6 is één van je experimenteel bepaalde waardes vermoed
  ik.
  Als je wil kan ik laten zien van waar deze komt op de
  technische tekeningen (die ik heb gekregen voor het freeswerk).
  En dan is het +7 of +8 dacht ik.
  
  Maar +6 is zeker goed genoeg!

Revision Comment by Matthias Claessens on 27 August 2025, 07:50 
http://subversion02:8060/cru/CR-2895#c4510

  Kleintje, nu is deze comment twee keer aanwezig.
  Zou je kunnen toevoegen of het om de rug of bodem gaat?
  Idem voor Rechts uiteraard.

Revision Comment by Matthias Claessens on 27 August 2025, 07:55 
http://subversion02:8060/cru/CR-2895#c4511

  Hier ga ik dezelfde feedback geven dat ik van Wim heb ontvangen
  over dezelfde component.
  Door deze variabelen te gebruiken lijk je een verband te leggen
  tussen deze cutout en de variabele.
  Dit terwijl er geen verband is.
  
  Persoonlijk ben ik ook fan van mijn cutouts iets groter te
  maken dan minimum nodig (in richtingen die er niet toe doen.)
  Zo gebruik ik:
  {code}
  "size": {
       "x": "CutOutData_CapZ1_Width + 18",
       "y": 500,
       "z": 500
  }
  {code}
  
  
  Zo is duidelijk dat enkel in de X-richting er een
  afhankelijkheid is van een andere variabele.
  Als ik het niet duidelijk genoeg uitleg kunnen we samen nog
  eens met Wim praten om zijn standpunt duidelijk te laten maken.


---

ID: CR-2895 http://subversion02:8060/cru/CR-2895

Title: [ICT-5151] 3D-MVX: Sifonkappen
Statement of Objectives:


State: Closed

Summary: 

Author: Joannes Lemmens
Moderator: Joannes Lemmens
Reviewers: (0 active, 1 completed*)
     Matthias Claessens   (*)