Opened 7 years ago
Closed 7 years ago
#4531 closed defect (fixed)
Tiny Corsica invalid vertex position
Reported by: | bb | Owned by: | elexis |
---|---|---|---|
Priority: | Must Have | Milestone: | Alpha 22 |
Component: | Maps | Keywords: | |
Cc: | Patch: | Phab:D384 |
Description
in r19444 the seeds 8776 and 4533 the loading of tiny corsica in Atlas fails with:
ERROR: JavaScript error: uncaught exception: getHeight: invalid vertex position (-6, 70) ERROR: CMapGeneratorWorker::Run: Failed to load RMS 'maps/random/corsica.js' WARNING: JavaScript warning: Script value conversion check failed: v.isNumber() (got type undefined)
(the values in getHeights can vary a bit)
This error was also present in A21, but at other seeds f.e. 7125 (tiny)
Change History (3)
comment:1 by , 7 years ago
Summary: | Corsica invalid vertx position → Corsica invalid vertex position |
---|
comment:2 by , 7 years ago
Milestone: | Backlog → Alpha 22 |
---|---|
Patch: | → Phab:D384 |
Summary: | Corsica invalid vertex position → Tiny Corsica invalid vertex position |
Note:
See TracTickets
for help on using tickets.
The error has probably been in there since the beginning of the map and probably only applies to tiny maps.
It's for-loop after
var nbBeaches = scaleByMapSize(2,5);
that creates the beach where players can build docks, specificallystraightPassageMaker(startX, startZ,endX,endZ, 25, 18, 4,clShore,null);
I've posted the patch that fixes it to https://code.wildfiregames.com/D384 There are still other things wrong with tiny maps on corsica, since the map wasn't exactly designed for that mapsize, but if people want to go for that map, they should be able to succeed the mapgen.