Skip to content
This repository has been archived by the owner on Oct 2, 2020. It is now read-only.

Geometry check #186

Open
5 tasks
easyw opened this issue Dec 4, 2017 · 7 comments
Open
5 tasks

Geometry check #186

easyw opened this issue Dec 4, 2017 · 7 comments
Labels
Enhancement Improves existing package 3D in the library

Comments

@easyw
Copy link
Contributor

easyw commented Dec 4, 2017

Hi,
if FreeCAD (release 0.17 > 12703) can be included in travis checking chain, it would be possible to check geometry of STEP models in an automatic way:

  1. modify the following command checking script to give positive or negative results to travis
    checkBOPnoGui.py.zip

  2. launch the following command to run the checking in command mode (without the need of a gui)
    freecadcmd checkBOPnoGui.py "full_path_to_STEP_file/resistor-wrong-BOP.step"
    or in linux
    freecadcmd-daily checkBOPnoGui.py "full_path_to_STEP_file/resistor-wrong-BOP.step"
    Here a wrong STEP model to test:
    resistor-wrong-BOP.zip

It would be possible then to put some checking at the PR template like:

  • is your 3D model made in a mechanical modeler?
  • have you submitted the source of the model (source file or script)?
  • have you submitted both STEP in scale 1:1 and wrl in scale 1:2.54?
  • are your 3D models aligned to the KiCad footprint?
  • is your WRL model using material properties?
@SchrodingersGat
Copy link
Contributor

Thanks! I have added a PR template based on your points above. I'll look into the FreeCad script soon

@easyw
Copy link
Contributor Author

easyw commented Dec 4, 2017

thx...
I forgot to add an other useful point:

  • have you added a screenshot of the 3D model(s) over the footprint(s) in KiCad in the comments of the PR?

@poeschlr
Copy link
Collaborator

poeschlr commented Dec 4, 2017

I think this is a related issue: KiCad/kicad-library#1546 (comment)

@Shackmeister
Copy link
Collaborator

Perhaps a Travis check if the model is referenced anywhere?

@SchrodingersGat
Copy link
Contributor

Do you mean referenced by a footprint?

@pointhi pointhi added the Enhancement Improves existing package 3D in the library label Jan 5, 2018
@Shackmeister
Copy link
Collaborator

@easyw
just noticed this issue again :)
do you believe the check could be implemented relatively painless into Travis?

@easyw
Copy link
Contributor Author

easyw commented Jul 11, 2018

No, unfortunately geometry check does not give a clear result. There could be many false positive results. I wouldn't add any check like this on Travis.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Enhancement Improves existing package 3D in the library
Projects
None yet
Development

No branches or pull requests

5 participants