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]

Submitter:  Martin Guy <martinwguy>
Submitted:  Wed 30 May 2007 12:52:10 PM UTC
   
 
Category:  renderer Severity:  1 - Wish
Release:  None Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

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>
Group 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 Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by strk (Posted a comment)
  • -email is unavailable- added by martinwguy (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2007-05-30 martinwguy Reassign ItemGnash - The GNU Flash player, bug #20008 Gnash - The GNU Flash player, task #6955

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code