nightstyles / alembic

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

AbcCoreAbstractTests reads random memory when getting cyclic and acyclic properties #56

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
This test does very basic testing of property values.
Upon reading, we see what looks like random memory
e.g. 
acyclic 5.62648e-317
cyclic 5.12975e-317
cyclic 2.122e-314
cyclic 5.32887e-317

Note, this is on the main branch. This bug might have the same root cause as 
other bugs that are entered about valgrind errors
---------------------------------------------------
zen AbcCoreAbstract/Tests> ./AbcCoreAbstractTests

Found my last lonely child, named 
//ABC/archiveChild/archiveChildChild/archiveChildChildChild
It has 3 sub-properties.
Found secondArrayProp
aProp has 1 samples.
Found scalarProp
sProp has 1 samples.
Found intArrayProp
scalarProp at sample 0 has the value 42

At Sample 0, secondArrayProp has 2 points, with the values:
0: 10639808, 1: 0,

Starting WriteScalarTest
Ending WriteScalarTest
Starting ReadScalarTest
Ending ReadScalarTest
Starting TimeSamplingScalarTest
acyclic 0
acyclic 5.62648e-317
acyclic 5.34873e-317
acyclic 5.62081e-317
acyclic 5.34127e-317
acyclic 5.31417e-317
acyclic 5.34901e-317
uniform 5.26791e-317
uniform 3
uniform 6
cyclic 5.12975e-317
cyclic 2.122e-314
cyclic 5.32887e-317
cyclic 0.5
cyclic 0.5
cyclic 0.5

Original issue reported on code.google.com by ard...@gmail.com on 16 Nov 2010 at 11:01

GoogleCodeExporter commented 9 years ago
This has been fixed in joetmp.

Original comment by ard...@gmail.com on 16 Nov 2010 at 11:01

GoogleCodeExporter commented 9 years ago
Hey Brian, this seems to be fixed; can you verify and set status as "Verified"?

Original comment by ard...@gmail.com on 16 Nov 2010 at 11:01

GoogleCodeExporter commented 9 years ago
I can no longer find  AbcCoreAbstractTests, so I guess in that sense this bug 
is no longer reproducible. Joe, if you still have a branch with this file could 
you update this entry with a pointer to its locaiton? Maybe I missed the test 
(though I did do a find for it)
thanks

Original comment by ard...@gmail.com on 16 Nov 2010 at 11:01

GoogleCodeExporter commented 9 years ago
In the branch joesperiment, there is a test, 
lib/Alembic/AbcCoreAbstract/Tests/OctessenceBug38.cpp, that verifies this bug 
is fixed.

Original comment by ard...@gmail.com on 16 Nov 2010 at 11:01