bugGnash - The GNU Flash player - Bugs: bug #20008, Reassigned to another tracker...

 
 

bug #20008: Reassigned to another tracker [was: Cache matrix operations to save FP calculation]

Submitted by:  Martin Guy <martinwguy>
Submitted on:  Wed 30 May 2007 12:52:10 PM UTC  
 
Category: rendererSeverity: 1 - Wish
Release: NoneStatus: None
Privacy: PublicAssigned to: None
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Wed 30 May 2007 08:29:45 PM UTC, comment #2:

THIS ITEM WAS REASSIGNED TO TASK #6955.

Please, do not post any new comments to this item.


Martin Guy <martinwguy>
Wed 30 May 2007 02:17:17 PM UTC, comment #1:

This would be a "task", not a "bug", anyway thanks for adding a reminder.

Sandro Santilli <strk>
Project Member
Wed 30 May 2007 12:52:10 PM UTC, original submission:

THIS ITEM WAS REASSIGNED TO TASK #6955.

Wish item: reduce FP consumption by caching matrix parameters instead of recalculating them

de Eric Hughes <eh@narthex.us>
per a gnash-dev <gnash-dev@gnu.org>
data 29/05/2007 18:38

when I looked at the
matrix code last month I did notice one thing I was going to recommend:
storing both canonical matrix parameters and the matrix itself and
computing the matrix only when necessary. In other words, treating the
matrix values as a cache. When matrix canonical values are assigned, all
the trigonometric function are recomputed, which is going to be the most
expensive operation of the lot.

Perhaps better: when assigning canonical values, check them against
existing canonical values. If the same, do nothing. If different, update
the value and invalidate the matrix values. When performing a matrix
operation, check validity first and recompute if necessary.

In the same vein, caching the sine and cosine of the canonical rotation
value means that changing only the scale wouldn't incur any new
trigonometric computation. In such a case the multiplications would dominate.

I don't know enough about how the matrix computations fit within the
overall control scheme of the player, but if they're in anything like an
inner loop, caching these calculations should significantly improve
performance.

Martin Guy <martinwguy>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by strk (Posted a comment)
  • -unavailable- added by martinwguy (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):

     

     

    Follows 1 latest change.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 30 May 2007 08:29:45 PM UTCmartinwguyReassign ItemGnash - The GNU Flash player, bug #20008=>Gnash - The GNU Flash player, task #6955

    Back to the top


    Powered by Savane 3.1-cleanup1