Skip to content
This repository has been archived by the owner on Nov 26, 2022. It is now read-only.

Not supported map import where several map reference to 1 tileset #12

Open
MetLob opened this issue Aug 10, 2015 · 3 comments
Open

Not supported map import where several map reference to 1 tileset #12

MetLob opened this issue Aug 10, 2015 · 3 comments

Comments

@MetLob
Copy link
Contributor

MetLob commented Aug 10, 2015

For example for solutions, i implement in my Map Editor additional import options:

  • use exist tiles
  • add tiles to end
  • insert tiles to position (by tile index): if (add tiles to end and use exist tiles not checked)

Implementation not hard, but its problem looked in every second game...

@pleonex
Copy link
Owner

pleonex commented Aug 10, 2015

Can you give some file path that have that problem or upload an image?

@pleonex pleonex added the bug label Aug 10, 2015
@MetLob
Copy link
Contributor Author

MetLob commented Aug 10, 2015

Hi, in Nitro formats i find this problem in Coraline game:
http://www.coreline.ru/files/Multimap.zip

Also, in customs or raw formats in several games: Pokemon (Platinum etc), Magical Starsign, Ys I DS (JP).
Example on Ys game, there all data in RAW nitro formats without headers:

  1. Select palette (also there problem from issue Not supported importing map images with multiple palettes #11)
    sample1 - palette selected
  2. Select tiles
    sample2 - tiles selected
  3. First map (also other maps in files from options.chr_5 to options.chr_12)
    sample1 - first map
  4. Another map
    sample1 - another map

@pleonex pleonex added enhancement and removed bug labels Aug 12, 2015
@pleonex
Copy link
Owner

pleonex commented Aug 12, 2015

I see the problem now, in fact it's another feature that I implemented in NinoImager.
Since Tinke is user-interactive and we don't know how many image will be imported we can't implement the same algorithms as NinoImager (an automatic importer).

The quick algorithm I can think of is: for each tile we would search it in the current image, if it does not exist we add it. And in any case do not remove unused tiles. Maybe some tiles won't be used anymore, but we can't know that since we don't know what map file will use an image file.

@pleonex pleonex added the plugin label Aug 12, 2015
@pleonex pleonex added this to the Version 0.9.1 milestone Aug 12, 2015
@pleonex pleonex modified the milestones: v0.9.2, Version 0.9.1 Mar 24, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants