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.

Sign up for GitHub

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

Jump to bottom

0.5.1 - animated forest tiles from OpenGFX+ Industries disappear #140

Closed
jrook1445 opened this issue May 11, 2020 · 2 comments
Closed

0.5.1 - animated forest tiles from OpenGFX+ Industries disappear #140

jrook1445 opened this issue May 11, 2020 · 2 comments
Labels
bug Something isn't working

Comments

@jrook1445
Copy link

jrook1445 commented May 11, 2020 •
edited

I'm using the animated forest industry tiles from OpenGFX+ Industries v6392 in Wannaroo Basic Industries.

On 0.5.0, all is ok, behaviour is as expected.

On 0.5.1, the sprites have disappeared and the forest industry is no longer visible in the main window.

I don't claim to understand how the animation code actually works, so unsure if something has been deprecated?

steps to reproduce: test newgrf attached

Linux Mint 19.3
nml 0.5.0, 0.5.1
PIL: 5.4.1
PLY: 3.11
python 3.5.2 (pyrun 2.2.3)
OpenTTD: 1.10.1

ogfxinds-forest.tar.gz

Edit - amended test newgrf
@FLHerne
Copy link
Contributor

FLHerne commented May 11, 2020

Nothing is supposed to have been deprecated in 0.5.1; this must be a bug of some kind.

@glx22 glx22 added the bug Something isn't working label May 12, 2020
@glx22
Copy link
Contributor

glx22 commented May 12, 2020

Found the issue, Working on it.

glx22 added a commit to glx22/nml that referenced this issue May 12, 2020
@glx22
Fix OpenTTD#140: non-advanced sprite layouts don't have flags
1602399
glx22 added a commit to glx22/nml that referenced this issue May 12, 2020
@glx22
Add: Regression test cases for OpenTTD#140
2fb3f68
@glx22 glx22 closed this as completed in 1dfa776 May 13, 2020
glx22 added a commit that referenced this issue May 13, 2020
@glx22
Add: Regression test cases for #140
7549330
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@glx22 @FLHerne @jrook1445