Unintended installation of `results` and `tests` folders with Choreographer

[hirohira9119]

Installing Choreographer via pip results in additional top-level folders under site-packages: results and tests. These are installed alongside the expected choreographer and choreographer.dist-info folders.

Could you clarify:

• Is this installation behavior intentional?
• If so, would it be possible to rename the folders to something more Choreographer-specific (e.g., choreographer.results, choreographer.tests) to avoid potential namespace confusion?
• If not intentional, could these folders be excluded from installation?

If this is already covered somewhere in the documentation and I’ve simply overlooked it, feel free to point me in the right direction.

Thanks in advance for your insights!

P.S. Thanks for the release of version 1.2; it's working smoothly in my environment without Chrome but with MS Edge, which has been very helpful.

[ayjayt]

That's a good question, let me take a look at it. Distribution semantics in python aren't great yet.

Happy to hear about 1.2.

[ayjayt]

It's likely cause we have namespaces enabled in our build system, so the build probably considers these folders, which have no __init__.py, but may have .py, to be partial packages.

That's unfortunate behavior, and I have to explore the issue a little bit to understand how to toggle it (and understand why I turned namespaces on in the first places, which I think is because I was fighting with including data files a couple years ago).

[hirohira9119]

Thank you for your comment. I understand that including the tests and results folders was not the intended behavior. I’ve opened PR #268 with a proposed fix to ensure only the choreographer package is installed. Would this approach work?