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

Regression in Table Autotiles (RMVX) #126

Open
sorlok opened this issue Jul 18, 2015 · 11 comments
Open

Regression in Table Autotiles (RMVX) #126

sorlok opened this issue Jul 18, 2015 · 11 comments

Comments

@sorlok
Copy link

sorlok commented Jul 18, 2015

Issue 107 fixed a problem with table autotiles appearing below grass tiles. Now, however, table legs appear above some tiles they shouldn't.

Original issue:
http://i.imgur.com/Monioxs.png

After Issue 107 fix:
http://i.imgur.com/vYmYUoM.png

How it should actually look (RMVX):
http://i.imgur.com/fSEc343.png

The actual tiles used here are circled below:
http://i.imgur.com/7Tqr3ti.png

And here's a sample project:
https://drive.google.com/file/d/0B1P7NepPcOslVFJHYzNzMHpTUmM/view?usp=sharing

@Ancurio
Copy link
Owner

Ancurio commented Jul 18, 2015

That's terrifying.

@Ancurio
Copy link
Owner

Ancurio commented Jul 18, 2015

The tiles that are implicitly placed above the table legs even though they're on the same layer are of A4 (Walls) type. Of course that interaction is not documented in the reference.. would you mind testing if there are any other tile types that occlude table legs when placed below? I have already written a fix for the corner case you outlined, but it would be nice to fix them all at once.

@sorlok
Copy link
Author

sorlok commented Jul 18, 2015

Sure, I'll try all the default tiles and get back to you.

@sorlok
Copy link
Author

sorlok commented Jul 18, 2015

Here's all of Tile Set A, in the same order as in the palette.

tiles_all

@Ancurio
Copy link
Owner

Ancurio commented Jul 19, 2015

Thanks. Can you send me the map.rvdata for this?

@sorlok
Copy link
Author

sorlok commented Jul 19, 2015

@sorlok
Copy link
Author

sorlok commented Jul 22, 2015

There's another problem with table legs that is probably easier to fix at the same time:

RPG Maker VX:
http://i.imgur.com/IyiGZe4.png

MKXP:
http://i.imgur.com/7sDRdL8.png

Note that the table legs on the very first row are not shown at all, because the tile they are connected to is not in the Viewport.

@Ancurio
Copy link
Owner

Ancurio commented Jul 22, 2015

It's been fixed already: 4d97a17

@sorlok
Copy link
Author

sorlok commented Jul 22, 2015

Ah, fantastic! I'll merge it and check it the next time I reboot to Linux.

Thanks!

@sorlok
Copy link
Author

sorlok commented Jul 31, 2015

@Ancurio Just checked, and 4d97a17 indeed fixes the second problem. Any update on the first problem?

@sorlok
Copy link
Author

sorlok commented Aug 13, 2015

I was able to dig into this a bit more, and I have a provisional fix:
29b7588

There are a few things that could be improved upon in this solution:

  1. I think that the inner check (line 640) is actually not needed; we just need to process "special" tiles in reverse order.
  2. I'm certainly missing something in the isSpecial() function, since I am not so familiar with RPG Maker tileset numbering.

Regardless, this hack fixes my table leg problems.

Splendide-Imaginarius added a commit to Splendide-Imaginarius/mkxp that referenced this issue Jan 13, 2024
…facts-license

Add license notice to GitHub Actions artifacts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants