Restructuring#16
Open
naushir wants to merge 5 commits into
Open
Conversation
Rename the lightbox registry's generic names to device-specific ones (DRIVERS -> LIGHTBOX_DRIVERS, register_driver -> register_lightbox_driver) and move the lightbox CLI from the package __main__ into lightbox_cli.py, so a second device kind can sit alongside it with parallel naming.
Relocate the lightbox device API out of ctt.devices into a standalone, top-level devices/ package (import name `devices`), so device control is a sibling of the core CTT library rather than nested inside it. Imports become `from devices import ...`, the ctt-lightbox script points at devices.lightbox_cli, and setuptools discovery/package-data are updated to match.
…apps/ Move the core ctt package (algorithms, core, data, detection, output, platforms, utils) from apps/ctt/ctt up to a top-level ctt/, and move the CLI shell into a small ctt_cli package (ctt.py -> apps/ctt_cli/cli.py) that imports the library absolutely. The `ctt` console script (ctt_cli.cli:main) is unchanged, so `ctt` on the command line still works.
Point the docs and a couple of stale code comments at the new structure: the device API lives in the top-level devices/ package (not ctt.devices), the CLI is ctt_cli, and the config example and udev rule paths drop the apps/ctt/ctt prefix. No functional change.
Drop the redundant apps/ctt-server wrapper directory so the ctt_server package lives directly at apps/ctt_server, matching apps/ctt_cli. The apps/ discovery root already covers it, so packages.find no longer needs the apps/ctt-server entry; the import name and the ctt-server console script are unchanged.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.