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

Fix crash when saving tilemap's cel image #4089

Merged
merged 1 commit into from Nov 2, 2023

Conversation

martincapello
Copy link
Member

After applying this PR:

  1. A script can save a cel's image from a tilemap layer without crashing Aseprite.
  2. The saved image is the rendered image using the tileset tiles for each tilemap index.
  3. A new lua test is added.

Fix #4069

@dacap
Copy link
Member

dacap commented Oct 11, 2023

Didn't review this, waiting for @Gasparoken, anyway it would be nice to add a note about any new sprite added in the tests/sprites/README.md file.

@Gasparoken
Copy link
Member

I'll review this today. Sorry for the delay.

@martincapello martincapello force-pushed the fix-save-tilemap-cel-image branch 2 times, most recently from 54da35a to 40f974b Compare October 12, 2023 16:51
@Gasparoken
Copy link
Member

Works great! I think the name MakeStdTilemap could be improved to describe:
"Make a blank sprite with Tilemap layer and specific Tileset"

@martincapello
Copy link
Member Author

martincapello commented Oct 17, 2023

Would MakeStdTilemapSpriteWithTileset work?

@Gasparoken
Copy link
Member

Would MakeStdTilemapSpriteWithTileset work?

LGTM!😊

@Gasparoken
Copy link
Member

I was testing and found that this solution saves an image with the size of the sprite (which could be different than the size of the image in the cel) and the image is placed at origin = 0.0. Therefore, it is necessary to shrink the image before saving, or define the correct size of the final image from the beginning.

Copy link
Member

@Gasparoken Gasparoken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🙌

@Gasparoken Gasparoken assigned dacap and unassigned Gasparoken Oct 18, 2023
@dacap dacap merged commit 3cfa5ef into aseprite:main Nov 2, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tilemap cel image saveAs call leads to segmentation fault.
3 participants