Skip to content

Skip comments in functional equivalence tests #2110

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jstone-lucasfilm merged 5 commits into from
Nov 14, 2024
+27 −3
Merged

Skip comments in functional equivalence tests #2110

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
af30893
Skip comment blocks when checking for equivalency.
kwokcb Nov 11, 2024
4227273
Make comment slightly broader
jstone-lucasfilm Nov 13, 2024
418758e
Merge branch 'main' into equiv_comments
jstone-lucasfilm Nov 14, 2024
2b40ab7
Remove unneeed new "comment" string, and existing `CommentElement::CA…
kwokcb Nov 14, 2024
af90935
Merge branch 'equiv_comments' of https://github.com/kwokcb/MaterialX …
kwokcb Nov 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 19 additions & 3 deletions source/MaterialXCore/Element.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -426,9 +426,25 @@ bool Element::isEquivalent(ConstElementPtr rhs, const ElementEquivalenceOptions&
}
}

// Compare children.
const vector<ElementPtr>& children = getChildren();
const vector<ElementPtr>& rhsChildren = rhs->getChildren();
// Compare all child elements that affect functional equivalence.
vector<ElementPtr> children;
for (ElementPtr child : getChildren())
{
if (child->getCategory() == CommentElement::CATEGORY)
{
continue;
}
children.push_back(child);
}
vector <ElementPtr> rhsChildren;
for (ElementPtr child : rhs->getChildren())
{
if (child->getCategory() == CommentElement::CATEGORY)
{
continue;
}
rhsChildren.push_back(child);
}
if (children.size() != rhsChildren.size())
{
if (results)
Expand Down
8 changes: 8 additions & 0 deletions source/MaterialXTest/MaterialXCore/Document.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -150,6 +150,9 @@ TEST_CASE("Document equivalence", "[document]")
unsigned int index = 0;
mx::ElementPtr child = doc->addNodeGraph("mygraph");
mx::NodeGraphPtr graph = child->asA<mx::NodeGraph>();
// Add comment block at the start of the first doc to check skipping
mx::ElementPtr comment = doc->addChildOfCategory(mx::CommentElement::CATEGORY);
comment->setDocString("Comment 1");
for (auto it = inputMap.begin(); it != inputMap.end(); ++it)
{
const std::string inputType = (*it).first;
Expand Down Expand Up @@ -205,6 +208,11 @@ TEST_CASE("Document equivalence", "[document]")
input->setName("input_" + inputType);
}
}
// Add comment blocks at end of second doc to check value and count checks
comment = doc2->addChildOfCategory(mx::CommentElement::CATEGORY);
comment->setDocString("Comment 2");
comment = doc2->addChildOfCategory(mx::CommentElement::CATEGORY);
comment->setDocString("Comment 3");

mx::ElementEquivalenceOptions options;
mx::ElementEquivalenceResultVec results;
Expand Down