More than three.

For code related discussions and questions
User avatar
Goth Zagog-Thou
Regular
Regular
Posts: 1582
Joined: 06 Jan 2007, 08:08
Location: Delta Base

Re: More than three.

Post by Goth Zagog-Thou »

Very good. All we need is the max tiles increase, then.

My changes:

map.cpp -- line 100

Code: Select all

// Maximum expected return value from get height
#define	MAX_HEIGHT			(1024 * ELEVATION_SCALE)
map.cpp -- line 132

Code: Select all

#define MAX_TERRAIN_TILES 255		// max that we support (for now)
map.h -- line 340

Code: Select all

#define TILE_MAX_HEIGHT (1024 * ELEVATION_SCALE)
Checking the code to make sure it compiles and plays nice.
User avatar
Rman Virgil
Professional
Professional
Posts: 3812
Joined: 25 Sep 2006, 01:06
Location: USA

Re: More than three.

Post by Rman Virgil »

.

I'm assuming you upped the height of mountains? If so, you may wanna check VTOL fly-over behavior.

.
.

Impact = C x (R + E + A + T + E)

Contrast
Reach
Exposure
Articulation
Trust
Echo
.
User avatar
Goth Zagog-Thou
Regular
Regular
Posts: 1582
Joined: 06 Jan 2007, 08:08
Location: Delta Base

Re: More than three.

Post by Goth Zagog-Thou »

Yep, gonna do that in a bit. I accidentally overwrote decals with 2.3.9 versions, so I'm fixing that at the moment. :oops:
User avatar
Goth Zagog-Thou
Regular
Regular
Posts: 1582
Joined: 06 Jan 2007, 08:08
Location: Delta Base

Re: More than three.

Post by Goth Zagog-Thou »

So far so good. :P There's no visible performance hit as of yet.

Next thing I have to do (after changing the VTOL flight height maximum) is compile a Master version of FlaME with the 'GUTS' support and get a quick test map made.

I'm getting tired though and it's been a long day, so I may do that tomorrow.
User avatar
NoQ
Special
Special
Posts: 6226
Joined: 24 Dec 2009, 11:35
Location: /var/zone

Re: More than three.

Post by NoQ »

Updated to v2.
  • Include changes just proposed by Goth Zagog-Thou.
  • Cycle the colors for map previews. Tileset 4 is previewed as arizona, tileset 5 is previewed as urban, etc.
  • Rely on tile types when guessing whether a tile is a water or a cliff, not on ground type names.
    • This allows to use multiple cliff textures on one tileset without encountering this problem.
    • This allows using multiple seabed textures on one tileset.
    • I don't yet understand how to make multiple water colors on one tileset, and how will they blend if it is made.
wz2100-20120706_093522-Mirage2-T1.jpg
You do not have the required permissions to view the files attached to this post.
Last edited by NoQ on 06 Jul 2012, 07:53, edited 2 times in total.
User avatar
NoQ
Special
Special
Posts: 6226
Joined: 24 Dec 2009, 11:35
Location: /var/zone

Re: More than three.

Post by NoQ »

Updated to v3.

Fixed oversized seabeds, fixed map preview color cycling (3%3!=3 lol).

Anything else? What should i do now? Should i post a ticket with this patch? :oops:

Note: the patch is still compatible with both master and 3.1_beta11!
User avatar
aubergine
Professional
Professional
Posts: 3462
Joined: 10 Oct 2010, 00:58

Re: More than three.

Post by aubergine »

FULL OF WIN!
"Dedicated to discovering Warzone artefacts, and sharing them freely for the benefit of the community."
-- https://warzone.atlassian.net/wiki/display/GO
User avatar
NoQ
Special
Special
Posts: 6226
Joined: 24 Dec 2009, 11:35
Location: /var/zone

Re: More than three.

Post by NoQ »

(after changing the VTOL flight height maximum)
How is it going? Do you have a sample map to try?

Probably another thing that needs to be tried is camera movement. So that it didn't dig into the ground on low views.
Anyway, i'm sure we had stretched heights in master at some point, but that was removed. Is anybody aware of this story?
User avatar
Rman Virgil
Professional
Professional
Posts: 3812
Joined: 25 Sep 2006, 01:06
Location: USA

Re: More than three.

Post by Rman Virgil »

.

Btw Goth - is this the one you are working with?: /* primitive 'bang- bang ' vtol height controller */ function.
.
.

Impact = C x (R + E + A + T + E)

Contrast
Reach
Exposure
Articulation
Trust
Echo
.
User avatar
Goth Zagog-Thou
Regular
Regular
Posts: 1582
Joined: 06 Jan 2007, 08:08
Location: Delta Base

Re: More than three.

Post by Goth Zagog-Thou »

Good morning fine people.

Great job, NoQ! I believe that covers everything. :D Loos great btw.

@Rman: Haven't checked yet. I went to bed early last night since I haven't slept much at all during the current heat wave. Had errands to run and bills to pay today, and more business to do later on this afternoon (Grandson's b-day party .. and yes, I'm somebody's grandfather :lol2: ). I probably won't get to it until tomorrow, so if anyone else cares to change the height, be my guest.
User avatar
Rman Virgil
Professional
Professional
Posts: 3812
Joined: 25 Sep 2006, 01:06
Location: USA

Re: More than three.

Post by Rman Virgil »

.

Gramps!! Wow. I thought only me and Kim were the anchient ones. ;)

I messed around with the function years ago trying to figure out how to deal with vtol speed bump stalls but I ain't doing anything like that anymore. Just FVMs work and taking into account these type changes in my Machinima component shoots. If this works out I'm visuslizing being able to do some hair-raising vertigo camera work. :D

.
User avatar
Goth Zagog-Thou
Regular
Regular
Posts: 1582
Joined: 06 Jan 2007, 08:08
Location: Delta Base

Re: More than three.

Post by Goth Zagog-Thou »

I think that can be arranged. :D
User avatar
Goth Zagog-Thou
Regular
Regular
Posts: 1582
Joined: 06 Jan 2007, 08:08
Location: Delta Base

Re: More than three.

Post by Goth Zagog-Thou »

/src/move.cpp:62

Code: Select all

/* max and min vtol heights above terrain */
#define	VTOL_HEIGHT_MIN				1024
#define	VTOL_HEIGHT_LEVEL			1150
#define	VTOL_HEIGHT_MAX				1225
That takes care of VTOL heights. :) I'll check for transporter heights, but I think those use the VTOL values above. They do.
User avatar
NoQ
Special
Special
Posts: 6226
Joined: 24 Dec 2009, 11:35
Location: /var/zone

Re: More than three.

Post by NoQ »

I think this shouldn't be altered at all. It's "above terrain", not "above zero". In vanilla VTOL_HEIGHT_MIN==250 is less than maximum terrain height. I didn't check though.
User avatar
Goth Zagog-Thou
Regular
Regular
Posts: 1582
Joined: 06 Jan 2007, 08:08
Location: Delta Base

Re: More than three.

Post by Goth Zagog-Thou »

So far it looks like it's 'above zero'. I'll test it more. At least all of the code is playing nice so far.

I haven't added your .patch in yet. Just how are these applied, anyway? I always wanted to know how that's done. :)