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

Upcoming tmxrasterizer features #3

Open
bjorn opened this issue Jan 9, 2024 · 2 comments
Open

Upcoming tmxrasterizer features #3

bjorn opened this issue Jan 9, 2024 · 2 comments

Comments

@bjorn
Copy link

bjorn commented Jan 9, 2024

I just extended the functionality of tmxrasterizer to be able to export multiple animation frames: mapeditor/tiled#3868.

It does not support creating a GIF though, so you might eventually want to update this tool to use the new --frames <n> and --frame-duration <ms> parameters. :-)

@lightlike
Copy link
Owner

That does sound interesting and convenient and is probably faster than loading the map over and over again. (especially for big maps)

I'll probably keep the old arguments around for some backwards compatibility, but I'll look into that when I have some time to write some code again.

@bjorn
Copy link
Author

bjorn commented Jan 9, 2024

That does sound interesting and convenient and is probably faster than loading the map over and over again. (especially for big maps)

Well, this comment made me realize that in my naive implementation, it was still loading the map over and over again. So it would likely only be faster due to avoiding launching multiple processes.

Anyway, now it loads the map only once, though in my benchmarks this made very little difference. Most of the time is spent encoding the frames as PNG, anyway. :-)

I'll probably keep the old arguments around for some backwards compatibility, but I'll look into that when I have some time to write some code again.

Yeah, either you'd have to keep the current approach for now or support both for compatibility reasons, so no rush at all.

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

No branches or pull requests

2 participants