Maya AbcImport has inconsistency behavior when checking if a poly mesh is animated or not

[GoogleCodeExporter]

What steps will reproduce the problem?
1. In Maya, AbcImport the attached file.

What is the expected output? What do you see instead?
Expected: The mesh should be imported properly
Actual:   The mesh is imported as an empty mesh.

What version of the product are you using? On what operating system?
1.0 on all platforms

Please provide any additional information below.

In CreateSceneHelper.cpp, 
CreateSceneVisitor::operator()(Alembic::AbcGeom::IPolyMesh& iNode):

    bool isConstant = iNode.getSchema().isConstant();

    // add animated poly mesh to the list
    if (!isConstant)
        mData.mPolyMeshList.push_back(iNode);

In MeshHelper.cpp, createPoly(double iFrame, Alembic::AbcGeom::IPolyMesh& 
iNode, MObject& iParent)

    // add other properties
    if (schema.getNumSamples() > 1)
    {
        MFloatPointArray emptyPt;
        MIntArray emptyInt;
        obj = fnMesh.create(0, 0, emptyPt, emptyInt, emptyInt, iParent);
        fnMesh.setName(name);
    }
    else
    {

(!schema.isConstant()) is not equivalent to (schema.getNumSamples() > 1)
I believe AbcImport should always check if a mesh is animated or not by 
isConstant()

Thanks!

Original issue reported on code.google.com by shinc...@gmail.com on 19 Aug 2011 at 2:38

Attachments:

• ht.abc

[GoogleCodeExporter]

Yup, you are correct, thanks!

Original comment by miller.lucas on 19 Aug 2011 at 4:48

• Changed state: Accepted

[GoogleCodeExporter]

http://code.google.com/r/millerlucas-dev/source/detail?r=4d01b5cb682927f34931f17
4e45473ed7bb25237

It will be included in the next release.  Thanks!

Original comment by miller.lucas on 25 Aug 2011 at 7:12

• Changed state: InProgress

[GoogleCodeExporter]

Original comment by ble...@gmail.com on 25 Aug 2011 at 10:50

• Added labels: Milestone-1.1

[GoogleCodeExporter]

Original comment by miller.lucas on 29 Aug 2011 at 5:03

• Added labels: Milestone-1.0.1
• Removed labels: Milestone-1.1

[GoogleCodeExporter]

Original comment by erik.str...@gmail.com on 6 Sep 2011 at 8:44

• Changed state: PleaseVerify

[GoogleCodeExporter]

Original comment by miller.lucas on 19 Sep 2011 at 9:32

• Changed state: Verified