Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#4275 closed defect (fixed)

[PATCH] Optimize Caledonian Meadows

Reported by: FeXoR Owned by: FeXoR
Priority: Should Have Milestone: Alpha 21
Component: Maps Keywords: patch
Cc: Patch:

Description

This optimizes the map Caledonean Meadows by:

  • Drastically decreasing load time (by a factor of ~20)
  • Paths should now be able to be used by siege engines
  • Adding load progress points
  • Slightly increased flattening at start locations

This should also fix load errors that where reported (Thus fixing a real defect).

After eccessive testing things that still can be improved:

  • Iberian walls can have trees in them (both if the wall is very irregular or if start location very close to high ground)
  • Not all resource spots have terrain texture

Attachments (6)

optimize_caledonian_meadows2016_10_9.patch (14.8 KB ) - added by FeXoR 7 years ago.
Patch as in ticket description
optimize_caledonian_meadows2016_10_9b.patch (14.9 KB ) - added by FeXoR 7 years ago.
Fixes as proposed by elexis, fixes an issue with an optional argument
optimize_caledonian_meadows2016_10_9c.patch (14.8 KB ) - added by FeXoR 7 years ago.
Some code cleanup
optimize_caledonian_meadows2016_10_9d.patch (14.7 KB ) - added by FeXoR 7 years ago.
Added a new variable to be easier to read
optimize_caledonian_meadows2016_10_9d.2.patch (14.7 KB ) - added by FeXoR 7 years ago.
Fuxed an error that can occurr when argument avoidClass in getPointsByHeight is undefined
optimize_caledonian_meadows2016_10_9e.patch (14.7 KB ) - added by FeXoR 7 years ago.
Just to let everybody know I'm stupid: Fixed a wronly placed ! and a let out of scope

Download all attachments as: .zip

Change History (9)

by FeXoR, 7 years ago

Patch as in ticket description

by FeXoR, 7 years ago

Fixes as proposed by elexis, fixes an issue with an optional argument

by FeXoR, 7 years ago

Some code cleanup

comment:1 by FeXoR, 7 years ago

Last patch applied changes proposed by Itms, thanks!

by FeXoR, 7 years ago

Added a new variable to be easier to read

by FeXoR, 7 years ago

Fuxed an error that can occurr when argument avoidClass in getPointsByHeight is undefined

by FeXoR, 7 years ago

Just to let everybody know I'm stupid: Fixed a wronly placed ! and a let out of scope

comment:2 by FeXoR, 7 years ago

Owner: set to FeXoR
Resolution: fixed
Status: newclosed

In 18819:

Optimizes random map Caledonian Meadows: Broadened paths, much faster generation, added progress indicators, larger bases. Reviewed by Itms. Fixes #4275

comment:3 by FeXoR, 7 years ago

Keywords: review removed
Note: See TracTickets for help on using tickets.