THERMAL CAMERA LEPTON: Bin not there when installed with pip

[iezqrom]

I wasn’t sure how to flag this one. In the repository, the bin folder exists and contains the code required for the thermal camera to work on Windows. However, after installing the package with pip, the bin folder isn’t present.

My suspicion: This might be related to where the bin folder is located. It’s currently outside of the main package structure, so it may not be included when the library is built or uploaded to PyPI.

What I did:

• Moved the bin folder inside poulet-py/.
• Updated the code to reference the new path accordingly.

Open question: There might be something in the build or upload process of poulet-py that needs to be adjusted so the bin folder is packaged and installed properly. I might be missing a configuration detail here.

[vkaramanis]

good catch.

there is a better way to include the folder in the pyproject.toml using either

[tool.hatch.build.targets.wheel.force-include]
"../bin" = "bin"

or

[tool.hatch.build.targets.wheel]
artifacts = [
  "bin/leptonUVC/x64/*.dll",
  "bin/leptonUVC/x64/*.xml
]

or both?
more info here

can you make this change?
maybe also to change the bin folder name to artifacts to have the same name convention with hatch?

[iezqrom]

I've implemented this, please review @vkaramanis

[vkaramanis]

I meant to try putting this folder outside poulet_py, but mapping it in pyproject.toml, so to not blend it with internal code.
Can you try this, moving artifacts to root folder and see if it works?

[iezqrom]

I meant to try putting this folder outside poulet_py, but mapping it in pyproject.toml, so to not blend it with internal code. Can you try this, moving artifacts to root folder and see if it works?

cool, i'll do next week too

[iezqrom]

I meant to try putting this folder outside poulet_py, but mapping it in pyproject.toml, so to not blend it with internal code. Can you try this, moving artifacts to root folder and see if it works?

it's working! please check, i had to made a few adjustments to one function, but working now with artifacts outside

let me know if it's ready for dev

@vkaramanis