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

--list-splices --split-layers --sheet creates duplicate data in meta.slices #1582

Closed
nphyx opened this issue Oct 1, 2017 · 1 comment
Closed
Assignees
Labels
Milestone

Comments

@nphyx
Copy link

@nphyx nphyx commented Oct 1, 2017

Possibly related to #1234.

When --list-slices is combined with --split-layers --sheet to create a combined atlas (as per https://www.aseprite.org/docs/cli/#export-each-layer-as-a-different-animation-in-the-same-sprite-sheet), the meta.slices array contains duplicate entries, one per slice per layer. The entries are completely redundant - they don't contain different keys entries per layer.

For example, with two layers "layer-1", "layer-2" and one slice, "my-slice", output would be:

{ 
"frames": { "mysprite (layer-1).ase":{...},"mysprite (layer-2).ase":{...}},
"meta":{ ...
    "slices":[
        {"name":"my-slice", ... "keys":[{/* key data for layer-1 */}]}
        {"name":"my-slice", ... "keys":[{/* key data for layer-1 */}]}
    ]}

}
Note that both slices contain key info for layer-1.
The two entries are completely redundant, and present the bounds data for the base layer. A desirable alternative would be something like:

{ ...
"meta":{ ...
    "slices": [
        {"name":"my-slice", ... "keys":[
            {"frame":0, bounds: {/* bounds for layer-1 */}},
            {"frame":1, bounds: {/* bounds for layer-2 */}}
        ]},
    ]
}

Example input/output: bugged.zip

To reproduce, use:

aseprite -b --list-slices --split-layers aseprite/bugged.ase --data bugged.json --sheet bugged.png

Aseprite and System version

  • Aseprite version: 1.2.2 Steam / Beta
  • System: Linux 4.12.8 / Arch
@johanwendin

This comment has been minimized.

Copy link

@johanwendin johanwendin commented Jan 1, 2018

same thing with --list-tags when combined with --split-layers, the tags will repeat (in order) once per layer.

@dacap dacap self-assigned this Oct 1, 2019
@dacap dacap added this to the v1.x-bugs milestone Oct 1, 2019
@dacap dacap closed this in ae9a812 Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.