Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restructure the repository. #9

Merged
merged 13 commits into from
Jul 5, 2024
Merged

Restructure the repository. #9

merged 13 commits into from
Jul 5, 2024

Conversation

kkeroo
Copy link
Collaborator

@kkeroo kkeroo commented Jul 4, 2024

This PR restructures the repository to follow the goal of publishing it.

  1. The source code is located in the folder depthai_nodes.
  2. requirements.txt is updated with a different depthai version which has wheels for all three OS (the previous version had only Linux wheel).
  3. Additionally, requirements-dev.txt file is added with requirements to commit and generate docs.
  4. pyproject.toml file is also added. Right now, python version is set requires-python = ">=3.8".
  5. Readme file is updated, but not complete.
  6. Also, cotributing file is added, currently without tests as there are none (should be added later).
  7. One GH workflow for pre-commit is added when pushing to main and dev branches. This workflow needs to be extended once tests and docs are added.

@kkeroo kkeroo requested a review from tersekmatija July 4, 2024 10:11
[project]
name = "depthai-nodes"
version = "0.0.1"
description = "Luxonis package providing Python nodes for on-host DepthAI model's output decoding and postprocessing."

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could just say "Community on-host nodes for DepthAI" or something along those lines.

@tersekmatija
Copy link

IMO this can be merged, let's also remove the empty nn folder.

@kkeroo kkeroo merged commit 46698eb into main Jul 5, 2024
1 check passed
@kkeroo
Copy link
Collaborator Author

kkeroo commented Jul 5, 2024

IMO this can be merged, let's also remove the empty nn folder.

This is already removed.

@kkeroo kkeroo deleted the restructure branch July 5, 2024 08:57
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.

2 participants