project-navigation
Personal tools

Author Topic: new tile for +oriental  (Read 13040 times)

Offline Kildor

  • Project Artist
  • Captain
  • ***
  • Posts: 757
  • Project mapper and some other stuff`er
    • View Profile
    • http://ufoai.nx0.ru
Re: new tile for +oriental
« Reply #15 on: September 05, 2011, 04:08:40 pm »
The main problem in round things is pathfinding and square game.

Offline jerikojerk

  • Squad Leader
  • ****
  • Posts: 149
    • View Profile
Re: new tile for +oriental
« Reply #16 on: September 06, 2011, 07:18:16 pm »
it's hard to understand without knowing the algorythm...

 i guess it can be a round thing with uncrossable objects in the square corner so we don't confuse the pathfinder.

edit: now i think i get it; pathfinder only know about 4 or 8 direction on a same level? if so round tower and square stair should fit ?
« Last Edit: September 06, 2011, 07:22:48 pm by jerikojerk »

Offline H-Hour

  • Administrator
  • PHALANX Commander
  • *****
  • Posts: 1923
    • View Profile
Re: new tile for +oriental
« Reply #17 on: September 06, 2011, 07:35:52 pm »
It could be done, but the circular staircase would have to be very large (at least 4x4 grid squares in-game) and there would be lots of wasted space. And it would be lots of work for what would be a slightly-annoying feature in-game.

Offline tilli

  • Rookie
  • ***
  • Posts: 72
    • View Profile
Re: new tile for +oriental
« Reply #18 on: September 14, 2011, 08:29:18 pm »
thx kildor for the changes

the idea with a larger tile is good, I will implement that later.

I changed the walls texture of the minaret a little.

Offline Mattn

  • Administrator
  • PHALANX Commander
  • *****
  • Posts: 4831
  • https://github.com/mgerhardy/vengi
    • View Profile
    • Vengi Voxel Tools
Re: new tile for +oriental
« Reply #19 on: September 14, 2011, 08:51:00 pm »
add them to master and integrate them

Offline H-Hour

  • Administrator
  • PHALANX Commander
  • *****
  • Posts: 1923
    • View Profile
Re: new tile for +oriental
« Reply #20 on: September 14, 2011, 10:07:43 pm »
Looks good!

Offline Mattn

  • Administrator
  • PHALANX Commander
  • *****
  • Posts: 4831
  • https://github.com/mgerhardy/vengi
    • View Profile
    • Vengi Voxel Tools
Re: new tile for +oriental
« Reply #21 on: October 03, 2011, 08:33:10 am »
your new map or_house_i is missing in the oriental.ump.

Offline tilli

  • Rookie
  • ***
  • Posts: 72
    • View Profile
Re: new tile for +oriental
« Reply #22 on: October 03, 2011, 09:21:09 pm »
Quote
your new map or_house_i is missing in the oriental.ump.

not any more, along with the other changes.

Offline Mattn

  • Administrator
  • PHALANX Commander
  • *****
  • Posts: 4831
  • https://github.com/mgerhardy/vengi
    • View Profile
    • Vengi Voxel Tools
Re: new tile for +oriental
« Reply #23 on: October 03, 2011, 09:22:31 pm »
https://github.com/ufoai/ufoai/commit/8ad98d14d0627d8ad9890ae3992a1c364859c597

i think you did something wrong - you readded the seedlist - and changed a lot of other things - i don't think that all changes were intentional. please doublecheck

Offline tilli

  • Rookie
  • ***
  • Posts: 72
    • View Profile
Re: new tile for +oriental
« Reply #24 on: October 04, 2011, 07:51:45 am »
Quote
you readded the seedlist
please explain what that means?

I only did the following:
changed order of entries, so same tile types (houses, stands, etc) are in the same block
added missing tiles
added tilesets
incorporated that in the maps

any consequences I'm not aware of?

Offline H-Hour

  • Administrator
  • PHALANX Commander
  • *****
  • Posts: 1923
    • View Profile
Re: new tile for +oriental
« Reply #25 on: October 04, 2011, 08:20:49 am »
Have a look at the link that mattn posted. It will give you a blow-by-blow account of what your commit changed in the .ump file. Anything with a red background and a (-) in the front is a line that was removed. Anything with a green background and a (+) in front was added.

From this you can see that your commit added a seedlist. Is it possible that you made your changes to what was not the latest copy of the file. You can see the file history as well.

Offline Mattn

  • Administrator
  • PHALANX Commander
  • *****
  • Posts: 4831
  • https://github.com/mgerhardy/vengi
    • View Profile
    • Vengi Voxel Tools
Re: new tile for +oriental
« Reply #26 on: October 04, 2011, 08:22:05 am »
See the diff i posted

Offline tilli

  • Rookie
  • ***
  • Posts: 72
    • View Profile
Re: new tile for +oriental
« Reply #27 on: October 04, 2011, 09:43:31 am »
got it, my mistake  :-[

will simply removing the seedlist be ok, or is more to do?

Offline H-Hour

  • Administrator
  • PHALANX Commander
  • *****
  • Posts: 1923
    • View Profile
Re: new tile for +oriental
« Reply #28 on: October 04, 2011, 12:18:51 pm »
will simply removing the seedlist be ok, or is more to do?

You should be able to see from the diff if there is any change there you didn't intend (other than the seedlist). Removing the seedlist is probably enough, but doing a quick load-check on each assembly after you've made the change would be nice just to ensure there are no errors accidentally introduced.

Offline tilli

  • Rookie
  • ***
  • Posts: 72
    • View Profile
Re: new tile for +oriental
« Reply #29 on: November 13, 2011, 01:10:03 pm »
finished some work on +oriental
- randomspawn
- added some buildings (houses, minaret, shed, stand)
- updated oriental.ump
- miscellaneous stuff

For the moment I would like to wrap it up, are there any issues, which need to be take care of in +oriental, which I'm not aware of?