Topic: pak128 Half Height Tiles

[Fabio]

I think it's about time to start merging back the branch into Pak128 trunk.
Half height conversion is at a fairly good stage and might benefit of nightly builds for testing.
I'll start working on it in the next days.

[Sarlock]

I was thinking about that just this morning.  I agree.  Nothing major has popped up with testing so far, you've done an amazing job.

[greenling]

Hello Fabio
Can you please post me per pm then you beginn with working on the pak128 Half Height Tiles?

[Fabio]

why?

if you want a last full backup of pak128 you can do it now, it won't change anymore until double height...

[sdog]

Greenling, you can also get all the old versions of the pak-source from github. It stores the complete history of changes.

https://github.com/simutrans/pak128/commits/master

just click on 'browse sourcecode' and you'll go to that old version.

[Fabio]

Ok the branch is now merged in trunk in r1304-1310.

Please test if anything is amiss and I'll remove the branch from svn.
I would also like some Linus user to test the updated makefile.

[greenling]

Hello Fabio
I Have make a Quickcheck with your Commits r1304-1310 they have all a bug,
 they are not more Crossover useably with the pak128.Britain Commit 1296.
Here A Photo out Pak128.britain r1298:

[Zeno]

I haven't got a lot of time, but I'll try after next nightly, which should be on tuesday. Does the make.bat/py (win) need to be tested too?

[greenling]

Hello Zeno
Fabio´s Commits r1304-1310 working not with the pak128.britain Commits 1296.
They Have wrong slopestiles.

[Fabio]

Please test them too, if you can. In my local copy they seem to work, but better if more can give me a feedback.

[isidoro]

I've tried it in Linux, but it fails to compile:

Code: [Select]

===> PAK128 factories
./makeobj PAK128 ./pak128/factories.pak factories/
Makeobj version 55.2 for Simutrans 112.4 Nightly and higher
writing file ./pak128/factories.pak
   reading file factories/brewery.dat
      packing factory.brewery
FATAL ERROR: Factory - Missing an indentification color! (mapcolor)
Aborting program execution ...

I've managed to compile a version of the old pak128.newlandscape with wine, a version of makeobj.exe and a modified pak128.bat and it worked fine, except for the error in pak128.bat I pointed out in the other thread.  However, rail bridge entrances seemed to be double height when joining tracks at the same level.  I'll check if it goes away if I manage to compile it with the Makefile.

[prissi]

makeobj was broken, please test the current version.

[Miziiik]

Are the icons missing? (I think that in branch icons was?) Or is it a bug?

[Fabio]

I think slope tools never had a dedicated cursor, but thank you for testing.

[greenling]

Hello Fabio
I have find a graphicbug in the pak128 commit 1310.


The line in the photo marked the new evelated railway .

[isidoro]

@prissi: now makeobj works and gives some warnings, but no errors.  Thanks.

@fabio: Makefile stops now with the following error:

Code: [Select]

===> ground.Outside.pak
./makeobj quiet PAK128 temp.pak landscape/grounds/
Makeobj version 55.2 for Simutrans 112.4 Nightly and higher
writing file temp.pak
   reading file landscape/grounds/marker.dat
      packing ground.Marker
   reading file landscape/grounds/slope.dat
      packing ground.Slopes
   reading file landscape/grounds/TextureGrounds.dat
      packing ground.LightTexture
      packing ground.ClimateTexture
      packing ground.ShoreTrans
      packing ground.SlopeTrans
      packing ground.Borders
   reading file landscape/grounds/basement.dat
      packing ground.Basement
   reading file landscape/grounds/sidewalk.dat
      packing misc.Sidewalk
   reading file landscape/grounds/fences.dat
      packing ground.Fence
   reading file landscape/grounds/water_ani.dat
      packing ground.Water
   reading file landscape/grounds/tunnel_ground.dat
      packing misc.TunnelTexture
./makeobj quiet EXTRACT temp.pak
Makeobj version 55.2 for Simutrans 112.4 Nightly and higher
  found 12 files to extract

  writing 'ground.Marker.pak' ...
  writing 'ground.Slopes.pak' ...
  writing 'ground.LightTexture.pak' ...
  writing 'ground.ClimateTexture.pak' ...
  writing 'ground.ShoreTrans.pak' ...
  writing 'ground.SlopeTrans.pak' ...
  writing 'ground.Borders.pak' ...
  writing 'ground.Basement.pak' ...
  writing 'misc.Sidewalk.pak' ...
  writing 'ground.Fence.pak' ...
  writing 'ground.Water.pak' ...
  writing 'misc.TunnelTexture.pak' ...
mv: cannot stat ‘ground.Outside.pak’: No such file or directory
make: *** [landscape/grounds] Error 1

And the relevant part of the Makefile is:

Code: [Select]

$(DIRGROUNDS):
        @echo "===> ground.Outside.pak"
        @mkdir -p $(PAKDIR)
        $(MAKEOBJ) quiet PAK128 temp.pak $@/
        $(MAKEOBJ) quiet EXTRACT temp.pak
        @rm temp.pak
        @mv ground.Outside.pak $(PAKDIR)/
        $(MAKEOBJ) quiet MERGE $(PAKDIR)/$(call make_name,$@) *.pak
        @rm *.pak

It says that it cannot find ground.Outside.pak when trying to move it to the PAKDIR directory.  DIRGROUNDS is directory landscape/grounds, and you have there:

Code: [Select]

TextureGrounds.dat  fences.dat  sidewalk.dat  tunnel_ground.dat
basement.dat        marker.dat  slope.dat     water_ani.dat


[sdog]

pak128's has had quite a strong pulse over the last three days:

1 author has pushed 8 commits to all branches, excluding merges. On master, 1,479 files have changed and there have been 25,284 additions and 21,757 deletions.

[Fabio]

@fabio: Makefile stops now with the following error:
[...]
It says that it cannot find ground.Outside.pak when trying to move it to the PAKDIR directory.

ground.Outside was moved to another location (alone in base folder).
makefile should work again in r1314

[isidoro]

Now Makefile doesn't give any errors, but:

• It produces a directory that isn't recognized by the ST exe.  Looking deeper, it still asks for a ground.Outside.pak that is missing.
• I managed to compile the pak in Linux with the python builder and it produces a working pakset, but the files are completely different from the ones produced by the Makefile (res buildings are joined in Makefile, but separate in python script, for instance).
• When comparing the files produced by new pak128 (python) against pak128.newlandscape, there are some (I guess intended) differences:

Code: [Select]

693,696d692
< cursor.Builder.pak
< cursor.GeneralTools.pak
< cursor.Marked.pak
< cursor.Mouse.pak
771a768
> factory.Corn_Farm.pak
825a823
> field.corn_field.pak
876,880c874,875
< menu.BarTools.pak
< menu.DialogeTools.pak
< menu.GeneralTools.pak
< menu.SimpleTools.pak
< misc.Construction.pak
---
> GUI.128.pak
> GUI.64.pak
891d885
< symbol.all.pak

Finally, when running old pak128.newlandscape it works, while new pak128 doesn't zoom in/out correctly (when zooming out, trees don't get smaller; when zooming in, tiles in the ground don't get bigger) and other funny effects.

[sdog]

I managed to compile the pak in Linux with the python builder and it produces a working pakset, but the files are completely different from the ones produced by the Makefile (res buildings are joined in Makefile, but separate in python script, for instance).

This is semi intentional in the Makefile. Ie I considered it not important enough to bother when i wrote the initial pak128 Makefile. Perhaps noone else did bother either since (that would be mostly neroden who fixed  the many mistakes i made.)

[VS]

pak128's has had quite a strong pulse over the last three days:

Splendid!

[Tazze]

Hello Fabio and all.
I updated ground tool icons refer to pak.Britain. Please test it.
http://simutrans-germany.com/files/upload/pak128_new_ground_tool_icon.zip

Sorry I forgot to write that Don't miss backup.

[Tazze]

Hello Fabio and all.
I updated ground tool icons refer to pak.Britain. Please test it.
http://simutrans-germany.com/files/upload/pak128_new_ground_tool_icon.zip

Sorry I forgot to write that Don't miss backup.Don't forget Buck-up file.

I succeeded in starting new game several times after put into it ,but, recently Fatal Error occur.
@pak128.open.r1328
FATAL ERROR:vector_tpl<T>::[] - Pc: index out bounds: 59 not in 0..58
Aborting program execution....

[prissi]

Should be fixed by todays nightly, it was a bug with city name generation.

[Tazze]

Hi prissi. Thank you for this.

To: Administer of Pak128
Can you commit this menuconfig.tab and the source of this tool bar icons , If  this Gui64.pak did not have bug.
Because I just fixed original icon.

[tonu]

Hi!
Will bus stations be able to be built in soft slopes? (only bus or tram, for other vehicles it don't seem to be logical)

[Fabio]

No stations can be built on any slope with current code.
Station revamping became a priority because tracks were lowered.


Sent from my iPhone using Tapatalk

[gauthier]

Where is exactly pak128, regarding half slopes, today ? I didn't have enough free time to follow this thread last months. Anyway I'd like to know if there are compiled releases of both ST and pak128 to try this feature.

[gauthier]

I finally got to read all the thread (not so long though). I'm a bit sad about seeing rail ballast flattened but I understand the pros that has. Updating SNFOS trains for this change won't be too hard, I would be able to achieve that quickly after the first stable release of both ST and pak128 with new landscapes.

Just a problem : for all ways, with backward half slopes : there's a line of missing pixels at the lower end of the slope and lines don't connect well between tiles (especially with roads).

Another problem, which is a code-related one, vehicles can pass under half-height bridges. Is that intended ?

[kierongreen]

Vehicles can pass under half height bridges if they exist - however  building half height bridges over ways, and building ways under half height bridges should both be forbidden. So this eventuality shouldn't happen.