blender/blender-asset-tracer
2
5
Fork 5
Code Issues 4 Pull Requests 1 Projects Releases Wiki Activity

Pointcaches are not found when stored on disk and sending to flamenco (Missing file warning) #57672

New Issue
Closed
opened 2018-11-07 10:45:40 +01:00 by Jeroen Bakker · 6 comments
Jeroen Bakker commented 2018-11-07 10:45:40 +01:00
Member
Copy Link

disk caches are stored in blendcache_<blendfilename_no_ext> folder.

when sending a file to flamenco the filename is changed from orig.blend to orig.flamenco.blend
See https://git.blender.org/gitweb/gitweb.cgi/blender-cloud-addon.git/blob/HEAD:/blender_cloud/flamenco/init.py#l314
But caches still remain in original location: //blendcache_orig/

When using a disk cache (PTCACHE_DISK_CACHE) the point cache location is reconstructed using the new filename resulting in the wrong location
In https://git.blender.org/gitweb/gitweb.cgi/blender-asset-tracer.git/blob/HEAD:/blender_asset_tracer/trace/modifier_walkers.py#l149

BAT wants to load the caches from //blendcache_orig.flamenco/ which is incorrect. As this folder does not exist the file sequence expander yields an empty list
https://git.blender.org/gitweb/gitweb.cgi/blender-asset-tracer.git/blob/HEAD:/blender_asset_tracer/trace/file_sequence.py#l45

Best way to solve it is not to use .flamenco.blend, but .flame or something similar. As PathLib.stem only removes the last extension. Btw blender strips the last 6 characters so use an extension of exactly 5 chars :-)
Or rename the extension back to .blend on the server
Something to think about

disk caches are stored in blendcache_<blendfilename_no_ext> folder. when sending a file to flamenco the filename is changed from orig.blend to orig.flamenco.blend See https://git.blender.org/gitweb/gitweb.cgi/blender-cloud-addon.git/blob/HEAD:/blender_cloud/flamenco/__init__.py#l314 But caches still remain in original location: //blendcache_orig/ When using a disk cache (PTCACHE_DISK_CACHE) the point cache location is reconstructed using the new filename resulting in the wrong location In https://git.blender.org/gitweb/gitweb.cgi/blender-asset-tracer.git/blob/HEAD:/blender_asset_tracer/trace/modifier_walkers.py#l149 BAT wants to load the caches from //blendcache_orig.flamenco/ which is incorrect. As this folder does not exist the file sequence expander yields an empty list https://git.blender.org/gitweb/gitweb.cgi/blender-asset-tracer.git/blob/HEAD:/blender_asset_tracer/trace/file_sequence.py#l45 Best way to solve it is not to use .flamenco.blend, but .flame or something similar. As PathLib.stem only removes the last extension. Btw blender strips the last 6 characters so use an extension of exactly 5 chars :-) Or rename the extension back to .blend on the server Something to think about
Jeroen Bakker commented 2018-11-07 10:45:40 +01:00
Author
Member
Copy Link

Added subscriber: @Jeroen-Bakker

Added subscriber: @Jeroen-Bakker
Jeroen Bakker commented 2018-11-07 10:55:09 +01:00
Author
Member
Copy Link

Added subscribers: @dr.sybren, @fsiddi

Added subscribers: @dr.sybren, @fsiddi
Sybren A. Stüvel commented 2018-11-07 13:08:34 +01:00
Member
Copy Link

Nice analysis, thanks.

Best way to solve it is not to use .flamenco.blend, but .flame or something similar. As PathLib.stem only removes the last extension. Btw blender strips the last 6 characters so use an extension of exactly 5 chars :-)
Or rename the extension back to .blend on the server

Adding an option to BAT? to rename the blend file while creating the BAT Pack shouldn't be too hard. That way we can keep the .blend suffix for blend files. The approach would then be:

  • Blender Cloud Add-on (BCA) still saves locally as orig.flamenco.blend
  • BAT? is told to copy orig.flamenco.blend to {job-storage-dir}/orig.blend
  • BCA uses orig.blend as filename in the Flamenco job JSON sent to the Flamenco Server.

@Jeroen-Bakker what are your thoughts on this approach?

Nice analysis, thanks. > Best way to solve it is not to use .flamenco.blend, but .flame or something similar. As PathLib.stem only removes the last extension. Btw blender strips the last 6 characters so use an extension of exactly 5 chars :-) > Or rename the extension back to .blend on the server Adding an option to BAT? to rename the blend file while creating the BAT Pack shouldn't be too hard. That way we can keep the `.blend` suffix for blend files. The approach would then be: - Blender Cloud Add-on (BCA) still saves locally as `orig.flamenco.blend` - BAT? is told to copy `orig.flamenco.blend` to `{job-storage-dir}/orig.blend` - BCA uses `orig.blend` as filename in the Flamenco job JSON sent to the Flamenco Server. @Jeroen-Bakker what are your thoughts on this approach?
Gary Ritchie commented 2020-09-01 18:07:20 +02:00
Copy Link

Added subscriber: @GaryRitchie

Added subscriber: @GaryRitchie
Sybren A. Stüvel commented 2022-02-28 13:17:45 +01:00
Member
Copy Link

Changed status from 'Confirmed' to: 'Archived'

Changed status from 'Confirmed' to: 'Archived'
Sybren A. Stüvel commented 2022-02-28 13:17:45 +01:00
Member
Copy Link

A new generation of Flamenco (version 3.0) is in development (in the main branch at https://developer.blender.org/diffusion/F/), so all issues with Flamenco version 2 or older will be archived.

A new generation of Flamenco (version 3.0) is in development (in the `main` branch at https://developer.blender.org/diffusion/F/), so all issues with Flamenco version 2 or older will be archived.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
legacy module
Platforms, Builds, Tests & Devices

  
legacy project
Blender Asset Tracer

  
legacy project
Platform: Windows

  
Priority::Normal

   
Status::Archived

   
Status::Confirmed

   
Status::Needs Triage

   
Status::Resolved

   
Type::Design

   
Type::Report

   
Type::To Do
No Label
legacy module
Platforms, Builds, Tests & Devices
legacy project
Blender Asset Tracer
legacy project
Platform: Windows
Priority::Normal
Status::Archived
Status::Confirmed
Status::Needs Triage
Status::Resolved
Type::Design
Type::Report
Type::To Do
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-asset-tracer#57672
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?