DDMAL / pixel_wrapper

Rodan Pixel.js Wrapper
Other
0 stars 1 forks source link

Possible conflict with renaming the "name" key and other jobs' compatibility #8

Closed EricHanLiu closed 6 years ago

EricHanLiu commented 6 years ago

If other jobs also rely on the name key being a specific value (eg. PNG - Layer 1 Input), could run into conflicts by renaming. We renamed to add clarity, such as which specific PNG type (rgba PNG - Layer 1 Input) is required as a resource.

EricHanLiu commented 6 years ago

Used docker cp to get jobs folder locally, and checked all jobs (including symlinked jobs) for occurrences of the specific name key (such as PNG - Layer 1 Output). None were found, so there shouldn't be an issue with renaming them. Note that this applies to the Calvo's classifier job as well.