Skip to content

Comments

Updated IMQCAM schemas, including new ULI form#9

Open
arachid1 wants to merge 10 commits intomainfrom
20250617
Open

Updated IMQCAM schemas, including new ULI form#9
arachid1 wants to merge 10 commits intomainfrom
20250617

Conversation

@arachid1
Copy link
Collaborator

@arachid1 arachid1 commented Jun 18, 2025

Questions for Brett (but can be answered by anyone):

  • location, Brand Model, Nozzle Type only take one value across all examples. Are there others? If so, should they be categorical variables?
  • FID = foreign ID? or file ID?
  • sanity check: plate thickness is different from layer thickness, correct?
  • You use 'thickness' to refer to the number of layers. We used 'layerThickness' to refer to the thickness in mm. Therefore, I changed 'thickness' (number of layers) to 'layerCount'. Is that something you're comfortable with? open for suggestions.
  • is hatch spacing and hatch offset the same or different?
  • what are the bounds on offset/restriction/rotation angles?

Unification:

  • I use the same definitions for laserSpeed, laserPower, hatchSpacing, and layerThickness. then duplicated them across all files.
  • ULI had an additional definition for layerCount, but subject to change.
    • I didn't look hard enough to see if other fields in the ULI form are eligible. for a new definition. let's do that before merging.
      NB: CMU's printer build use micrometers while Brendan Croom's NASA build uses milimeters for the same fields like hatchSpacing. shouldn't be a problem at ingest, since JSON schemas doesn't validate anything, but will have to be mindful in postprocessing.

@arachid1
Copy link
Collaborator Author

Build Importer v1.2 CMU01 (2).xlsx
Brett's file for reference

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant