FBX Import: Read known sequential bytes simultaneously #104823

Thomas Barlow · 2023-08-06T19:03:00+02:00

Thomas Barlow commented

2023-08-06 19:03:00 +02:00

The start of each element and the start of each array property have a
consistent series of values with fixed sizes. Reading these series of
values with a single read call and single unpack call is slightly faster
than reading them individually.

This reduces the time taken to parse fbx files to about 90% to 95% of
the original time. Array decompression makes up about half of the time
taken currently, but this could be multithreaded, in which case, this
patch would reduce the time to about 80% to 90% instead.

The start of each element and the start of each array property have a consistent series of values with fixed sizes. Reading these series of values with a single read call and single unpack call is slightly faster than reading them individually. This reduces the time taken to parse fbx files to about 90% to 95% of the original time. Array decompression makes up about half of the time taken currently, but this could be multithreaded, in which case, this patch would reduce the time to about 80% to 90% instead.

Thomas Barlow added 1 commit 2023-08-06 19:03:02 +02:00

FBX Import: Read known sequential bytes simultaneously 51c0bc94ef

The start of each element and the start of each array property have a
consistent series of values with fixed sizes. Reading these series of
values with a single read call and single unpack call is slightly faster
than reading them individually.

This reduces the time taken to parse fbx files to about 90% to 95% of
the original time. Array decompression makes up about half of the time
taken currently, but this could be multithreaded, in which case, this
patch would reduce the time to about 80% to 90% instead.

Thomas Barlow requested review from Bastien Montagne 2023-08-06 19:10:47 +02:00

Bastien Montagne approved these changes 2023-08-08 17:00:17 +02:00

Bastien Montagne merged commit a600aeb2e2 into main

2023-08-08 17:00:36 +02:00

Bastien Montagne deleted branch parse_sequential_bytes_pr

2023-08-08 17:00:36 +02:00

Bastien Montagne referenced this issue from a commit

2023-08-08 17:00:37 +02:00

FBX Import: Read known sequential bytes simultaneously

Sign in to join this conversation.

No reviewers

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

FBX Import: Read known sequential bytes simultaneously #104823