johnahartman / alembic

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

AbcImport doesn't accept long names filtering #312

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Run AbcImport -ft "/pCube1/pCubeShape1" test.abc

What is the expected output? What do you see instead?
It will not import anything as the regex system only uses
the short name of the polymeshes

Very simple fix :
//const MString name = parent.getName().c_str();
const MString name = parent.getFullName().c_str();

Or maybe the command needs an option to use or not full
paths when running the filters.

Original issue reported on code.google.com by skydiver...@gmail.com on 28 Aug 2013 at 3:28

GoogleCodeExporter commented 8 years ago
Passing this on to Barnaby to see if expanding this to the full name would 
interfere with any workflows.

Doing this in CreateSceneVisitor::previsit would work for both the include AND 
exclude patterns.

Original comment by miller.lucas on 28 Aug 2013 at 4:42

GoogleCodeExporter commented 8 years ago

Original comment by miller.lucas on 28 Aug 2013 at 9:27

GoogleCodeExporter commented 8 years ago
Fix was committed here, will be released in 1.5.1:

http://code.google.com/r/millerlucas-dev/source/detail?r=0612facbc727a40b86d1837
da96ce09cbaf1064c

Original comment by miller.lucas on 17 Sep 2013 at 8:44

GoogleCodeExporter commented 8 years ago

Original comment by miller.lucas on 19 Sep 2013 at 8:13