studio/blender-studio-tools
13
32
Fork 19
Code Issues 22 Pull Requests 4 Projects Releases 1 Activity

Remap: Refactor scripts #177

Merged
Nick Alberelli merged 17 commits from :feature/refactor-remap-script into main 2023-12-07 16:17:27 +01:00
Conversation 1 Commits 17 Files Changed 5 +1 -1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 104fb27eab - Show all commits

2
scripts/remap/README.md
View File

@ -8,4 +8,4 @@ This directory contains scripts that are useful for re-organizing production dir
3. Now you are ready to re-organize your mapped directory, move files into different folders, rename files and remove duplicates. 3. Now you are ready to re-organize your mapped directory, move files into different folders, rename files and remove duplicates.
4. Re-run the remap tool via `python -m remap` to update your map with the new file locations. The tool will print a bbatch, copy this for use in step 6. 4. Re-run the remap tool via `python -m remap` to update your map with the new file locations. The tool will print a bbatch, copy this for use in step 6.
5. Enter bbatch directory `cd blender-studio-pipeline/scripts/bbatch` 5. Enter bbatch directory `cd blender-studio-pipeline/scripts/bbatch`
6. Run provided bbatch command, similar to `python -m bbatch /my_files/ --script /{path_to_script}/remap_blender_paths_for_my_files.py --nosave --recursive` to update all references to the remapped directory contents in your .blend files. 6. Run provided bbatch command, similar to `python -m bbatch {my_files}/ --nosave --recursive --script {path_to_script}/remap_blender_paths.py --args "{path_to_json}/dir_map.json"` to update all references to the remapped directory contents in your .blend files.
TinyNick marked this conversation as resolved Outdated
Sebastian Parborg commented 2023-12-07 15:11:09 +01:00
Outdated
Review
Copy Link

Hmm, this command seems out of date?
The python file is not called remap_blender_paths_for_my_files.py anymore and it also takes the json file path as a command now.

I would also drop the leading slashed in /my_files/ and /{path_to_script}.

Also to make it consistent I would do convert my_files/ to {my_files}/ to indicate that this is something the user has to replace.

Hmm, this command seems out of date? The python file is not called `remap_blender_paths_for_my_files.py` anymore and it also takes the json file path as a command now. I would also drop the leading slashed in `/my_files/` and `/{path_to_script}`. Also to make it consistent I would do convert `my_files/` to `{my_files}/` to indicate that this is something the user has to replace.
Nick Alberelli commented 2023-12-07 16:15:59 +01:00
Outdated
Review
Copy Link

Thank you for pointing that out. I have updated the README with this commit 104fb27eab thank you!

Thank you for pointing that out. I have updated the README with this commit 104fb27eab223b16d1a2c50e24f240126b1d7c81 thank you!