Skip to content

Conversation

@gbeane
Copy link
Collaborator

@gbeane gbeane commented Sep 17, 2025

Note: branch name is misleading, should be rename-behavior-util

add cli utility for renaming a behavior in an existing JABS project

usage:

jabs-cli rename-behavior <project dir> <old behavior name> <new behavior name>

@gbeane gbeane self-assigned this Sep 17, 2025
@gbeane gbeane changed the title Rename behavior util Rename behavior utility Sep 17, 2025
Copy link
Contributor

@SkepticRaven SkepticRaven left a comment

Choose a reason for hiding this comment

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

I like the new is_valid_project_directory method. I see a lot of project folders inside project folders because someone went into the jabs folder.

@gbeane
Copy link
Collaborator Author

gbeane commented Sep 18, 2025

@SkepticRaven

I like the new is_valid_project_directory method. I see a lot of project folders inside project folders because someone went into the jabs folder.

Yeah, I need to expand its use now -- especially for any command line tool that expects to operate on an existing JABS project directory so we avoid creating jabs directories if the user passes an incorrect path. For the GUI, I want users to be able to open a directory that's never been opened in JABS before but we could prompt for confirmation if it's not already a JABS project directory (or maybe for the GUI it would be more useful to check that the selected directory actually contains video/pose pairs)

@gbeane gbeane merged commit 8edce34 into main Sep 18, 2025
2 checks passed
@gbeane gbeane deleted the rename-project-util branch September 18, 2025 22:01
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.

4 participants