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

Kmeans training step should check for numpy arrays #1682

Open
k034b363 opened this issue Mar 10, 2025 · 0 comments · May be fixed by #1683
Open

Kmeans training step should check for numpy arrays #1682

k034b363 opened this issue Mar 10, 2025 · 0 comments · May be fixed by #1683
Assignees
Labels
bugfix Bug fixes update Updates an existing feature/method
Milestone

Comments

@k034b363
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Currently, train.kmeans does not check the extensions of files in the input directory, so if intermediate or hidden files are added to the directory during workflow development, they sometimes cause problems.

Describe the solution you'd like
A check for each file that whatever gets read in results in a numpy array.

Describe alternatives you've considered
An alternative is to check the file extension of each file in the directory, but that seems like it would be a more complicated check than just checking if something is an array.

@k034b363 k034b363 added bugfix Bug fixes update Updates an existing feature/method labels Mar 10, 2025
@k034b363 k034b363 added this to the PlantCV v4.x milestone Mar 10, 2025
@k034b363 k034b363 self-assigned this Mar 10, 2025
@k034b363 k034b363 linked a pull request Mar 10, 2025 that will close this issue
8 tasks
@k034b363 k034b363 linked a pull request Mar 10, 2025 that will close this issue
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Bug fixes update Updates an existing feature/method
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant