search

johnahartman / alembic

Automatically exported from code.google.com/p/alembic
Other
0 stars 0 forks source link

Add a raw read method to ArrayProperty. #239

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Like the Investigation performance issues, this task eneds profiling data to 
back it up.
The idea though is to add two methods on properties so that client code could 
provide (and own) the memory buffers to read the sample into. Currently the 
ArrarySamplePtr returned by doing myArrayProperty.get (sampleSelector) is 
memory allocated and owned by alembic. The client application must copy out the 
data that it needs. For when the application could manage and use the data 
directly it sure would be nice if the application could provide the buffer and 
avoid a copy operation.  Also as part of this api, we might be able to leverage 
the float<>double converstion utils insid eof hdf5

Original issue reported on code.google.com by cookingw...@gmail.com on 8 Oct 2011 at 1:21

GoogleCodeExporter commented 8 years ago

Original comment by ble...@gmail.com on 8 Oct 2011 at 1:25

GoogleCodeExporter commented 8 years ago 
Changing the name of the ticket.

It boils down to:
ScalarPropertyReader has:
void getSample( index_t iSample, void *iIntoLocation )

ArrayPropertyReader should have something like this ALONG with:
void getSample( index_t iSample, ArraySamplePtr &oSample )

The precision stuff is already part of another ticket, but it could be part of 
this new function.

Original comment by miller.lucas on 15 Oct 2011 at 12:05

GoogleCodeExporter commented 8 years ago

Original comment by miller.lucas on 15 Oct 2011 at 12:05

GoogleCodeExporter commented 8 years ago

Original comment by miller.lucas on 15 Oct 2011 at 12:26

GoogleCodeExporter commented 8 years ago 
Done here:

http://code.google.com/r/millerlucas-dev/source/detail?r=aec8ba7db0f37ee36526acb
aa98e1723952c6d4f

Needs to be propagaged to Abc.

Original comment by miller.lucas on 12 Dec 2011 at 8:18

GoogleCodeExporter commented 8 years ago

Original comment by miller.lucas on 24 Jan 2012 at 1:41