patchWarzone 2100 Project - Patches: patch #895, Draw foundation beneath buildings

 
 
Show feedback again

You are not allowed to post comments on this tracker with your current authentification level.

patch #895: Draw foundation beneath buildings

Submitted by:  Per I. Mathisen <per>
Submitted on:  Sat 22 Dec 2007 03:07:18 PM UTC  
 
Category: FeaturePriority: 5 - Normal
Status: Wont DoPrivacy: Public
Assigned to: Per I. Mathisen <per>Open/Closed: Open
Planned Release: None

Mon 19 Oct 2009 09:20:04 PM UTC, comment #5:

New version of the patch, updated to most recent trunk, and now we set the height position of the defensive building based on where its outer radius intersects with the tile underneath, instead of simply levelling with the higher tile corner.

(file #7036)

Per I. Mathisen <per>
Project AdministratorIn charge of this item.
Mon 24 Dec 2007 09:26:40 AM UTC, comment #4:

I have not done a lot of profiling, and at this stage, I am actually not terribly concerned about performance. The renderer was written for 1990s hardware, and to do some of the things we will wan to do with it, it needs to be updated to stream data.

We already draw only what is needed. That is, a 64x64 view window of the whole map (much smaller window in the original, but it also had less zoom out).

There problem with deformable terrain is not the time it takes to deform the terrain (which is irrelevant), but that changing "compiled" data is hard to do and hard to maintain. See the render branch for how I do this for colours there for an example.

This patch was mostly created as part of an experiment of mine to figure out whether the stuff I do in the render branch is really necessary. I think it probably is.

But there is also a second problem - we will want static buildings. This would be possible if not for the fact that for each frame, some buildings are deformed on the fly, with their vertices manipulated. This is defensive buildings and buildings hit by eletronic warfare.

Per I. Mathisen <per>
Project AdministratorIn charge of this item.
Mon 24 Dec 2007 05:41:15 AM UTC, comment #3:

While it sort of works, the outcome is a bit mixed.

Have you profiled the actual time spent drawing the terrain?

I would think that it would be better to just modify the draw routine to draw only what is needed.

I think it is nicer to have terrain that can be deformed by whatever we come up with in the future, and since the game only does that for buildings now, I don't think the performance gain will be that great?

Bugs Buggy <buginator>
Project Administrator
Sun 23 Dec 2007 11:24:48 PM UTC, comment #2:

New version that actually works.

(file #3423)

Per I. Mathisen <per>
Project AdministratorIn charge of this item.
Sun 23 Dec 2007 12:25:17 PM UTC, comment #1:

This patch should show foundation only below defensive buildings. If it would only use the correct texture page...

(file #3419)

Per I. Mathisen <per>
Project AdministratorIn charge of this item.
Sat 22 Dec 2007 03:07:18 PM UTC, original submission:

In order to make buildings and terrain static, this patch adds a dynamically created foundation model between the building and the terrain underneath, instead of modifying the terrain when buildings are added.

Currently it has a bug that prevents the texture from showing and lots of debug code in it, but it should show the effect.

Per I. Mathisen <per>
Project AdministratorIn charge of this item.

 

Attached Files
file #7036:  foundation6.diff added by per (8kB - text/x-patch)
file #3423:  foundation3.diff added by per (7kB - text/x-patch)
file #3419:  foundation2.diff added by per (7kB - text/x-patch)
file #3414:  foundation1.diff added by per (14kB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by per (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 19 Oct 2009 09:20:04 PM UTCperAttached File-=>Added foundation6.diff, #7036
    Wed 06 Feb 2008 09:50:16 AM UTCperStatusIn Progress=>Wont Do
      Assigned toNone=>per
    Sun 23 Dec 2007 11:24:48 PM UTCperAttached File-=>Added foundation3.diff, #3423
    Sun 23 Dec 2007 12:25:17 PM UTCperAttached File-=>Added foundation2.diff, #3419
    Sat 22 Dec 2007 03:07:19 PM UTCperAttached File-=>Added foundation1.diff, #3414
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup