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

Reduce duplicate frames into one #1316

Closed
buckle2000 opened this issue Nov 4, 2016 · 6 comments
Closed

Reduce duplicate frames into one #1316

buckle2000 opened this issue Nov 4, 2016 · 6 comments

Comments

@buckle2000
Copy link

@buckle2000 buckle2000 commented Nov 4, 2016

Feature request:
Reduce duplicate frames into one frame in output and reference them in sheet data file.
This saves memory.

Aseprite and System version

  • Aseprite 1.1.7:
@oceanhahn

This comment has been minimized.

Copy link

@oceanhahn oceanhahn commented Nov 19, 2016

I'd actually like to see something like this as a way to link duplicate cels (frame-layer elements) on the timeline, if they're independent of each other. That way, duplicate frames (frames with complete sets of duplicate cels) would be optimized, but individual layers could still be modified my making specific cels unique later.

It would also make cel copying a bit more powerful, potentially. ASEprite already has cel instantiation between nonadjacent frames (!) so being able to have the software automatically find and link duplicate frames could be a really useful compliment to it.

Linked cels is not the same thing as linked frames, but I think it might be a more useful option.
I don't want to change this suggestion, though; it's just something it made me think of. ^^

@buckle2000

This comment has been minimized.

Copy link
Author

@buckle2000 buckle2000 commented Nov 20, 2016

@oceanhahn you should open a new issue for your idea.


By the way, do your mean that linked frames = multiple linked cells?

@oceanhahn

This comment has been minimized.

Copy link

@oceanhahn oceanhahn commented Nov 21, 2016

I'd interpreted "duplicate frames" as any pair of frames with perfectly identical pixel data between them when their layers are merged, but I may be misunderstanding you. ^^;

@amkk

This comment has been minimized.

Copy link

@amkk amkk commented Dec 7, 2017

hi @dacap
I want bump this because not yet labeled.

@dacap

This comment has been minimized.

Copy link
Member

@dacap dacap commented Dec 7, 2017

There are two things to improve:

  1. If we split layers, linked cels must be exported just once in the sprite sheet and JSON data should point to the same region
  2. If we don't split layers, and two (merged) frames are exactly the same, the data must be reused.

I guess this behavior should be optional (and on by default)

@dacap dacap added this to the v1.2 milestone Dec 7, 2017
@dacap dacap modified the milestones: v1.2, v1.3 Dec 11, 2018
@dacap dacap self-assigned this Oct 2, 2019
@dacap dacap modified the milestones: v1.3, v1.2 Oct 2, 2019
@dacap

This comment has been minimized.

Copy link
Member

@dacap dacap commented Oct 2, 2019

In next version (v1.2.16) -sheet-pack will merge duplicated images.

@dacap dacap closed this in 51be644 Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.