Refactored formula simplification with Clingo’s variants.

This commit is contained in:
Patrick Lühne 2017-03-23 00:44:10 +01:00
parent bf8cf75369
commit 825cd4de39
No known key found for this signature in database
GPG Key ID: 05F3611E97A70ABF
12 changed files with 296 additions and 261 deletions

3
.gitmodules vendored
View File

@ -4,6 +4,3 @@
[submodule "lib/catch"] [submodule "lib/catch"]
path = lib/catch path = lib/catch
url = https://github.com/philsquared/Catch url = https://github.com/philsquared/Catch
[submodule "lib/variant"]
path = lib/variant
url = https://github.com/mapbox/variant

View File

@ -253,13 +253,13 @@ struct Biconditional
struct Exists struct Exists
{ {
Exists(std::vector<VariablePointer> &&variables, Formula &&argument) Exists(std::vector<Variable> &&variables, Formula &&argument)
: variables{std::move(variables)}, : variables{std::move(variables)},
argument{std::move(argument)} argument{std::move(argument)}
{ {
} }
std::vector<VariablePointer> variables; std::vector<Variable> variables;
Formula argument; Formula argument;
}; };
@ -267,13 +267,13 @@ struct Exists
struct ForAll struct ForAll
{ {
ForAll(std::vector<VariablePointer> &&variables, Formula &&argument) ForAll(std::vector<Variable> &&variables, Formula &&argument)
: variables{std::move(variables)}, : variables{std::move(variables)},
argument{std::move(argument)} argument{std::move(argument)}
{ {
} }
std::vector<VariablePointer> variables; std::vector<Variable> variables;
Formula argument; Formula argument;
}; };
@ -332,7 +332,7 @@ Predicate deepCopy(const Predicate &other);
SpecialInteger deepCopy(const SpecialInteger &other); SpecialInteger deepCopy(const SpecialInteger &other);
String deepCopy(const String &other); String deepCopy(const String &other);
Variable deepCopy(const Variable &other); Variable deepCopy(const Variable &other);
std::vector<VariablePointer> deepCopy(const std::vector<VariablePointer> &other); std::vector<Variable> deepCopy(const std::vector<Variable> &other);
And deepCopy(const And &other); And deepCopy(const And &other);
Biconditional deepCopy(const Biconditional &other); Biconditional deepCopy(const Biconditional &other);
Exists deepCopy(const Exists &other); Exists deepCopy(const Exists &other);
@ -347,7 +347,7 @@ Term deepCopy(const Term &term);
std::vector<Term> deepCopy(const std::vector<Term> &terms); std::vector<Term> deepCopy(const std::vector<Term> &terms);
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
/*
const auto deepCopyUniquePtr = const auto deepCopyUniquePtr =
[](const auto &uniquePtr) -> typename std::decay<decltype(uniquePtr)>::type [](const auto &uniquePtr) -> typename std::decay<decltype(uniquePtr)>::type
{ {
@ -370,13 +370,27 @@ const auto deepCopyUniquePtrVector =
return result; return result;
}; };
*/
////////////////////////////////////////////////////////////////////////////////////////////////////
template<class Variant>
struct VariantDeepCopyVisitor
{
template<class T>
Variant visit(const T &x)
{
return deepCopy(x);
}
};
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
const auto deepCopyVariant = const auto deepCopyVariant =
[](const auto &variant) -> typename std::decay<decltype(variant)>::type [](const auto &variant) -> typename std::decay<decltype(variant)>::type
{ {
return variant.match([](const auto &x) -> typename std::decay<decltype(variant)>::type {return deepCopyUniquePtr(x);}); using VariantType = typename std::decay<decltype(variant)>::type;
return variant.accept(VariantDeepCopyVisitor<VariantType>());
}; };
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
@ -397,6 +411,22 @@ const auto deepCopyVariantVector =
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
const auto deepCopyVector =
[](const auto &vector) -> typename std::decay<decltype(vector)>::type
{
using Type = typename std::decay<decltype(vector)>::type::value_type;
std::vector<Type> result;
result.reserve(vector.size());
for (const auto &element : vector)
result.emplace_back(deepCopy(element));
return result;
};
////////////////////////////////////////////////////////////////////////////////////////////////////
inline BinaryOperation deepCopy(const BinaryOperation &other) inline BinaryOperation deepCopy(const BinaryOperation &other)
{ {
return BinaryOperation(other.operator_, deepCopy(other.left), deepCopy(other.right)); return BinaryOperation(other.operator_, deepCopy(other.left), deepCopy(other.right));
@ -474,9 +504,9 @@ inline Variable deepCopy(const Variable &other)
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
inline std::vector<VariablePointer> deepCopy(const std::vector<VariablePointer> &other) inline std::vector<Variable> deepCopy(const std::vector<Variable> &other)
{ {
return deepCopyUniquePtrVector(other); return deepCopyVector(other);
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////

View File

@ -3,9 +3,10 @@
#include <memory> #include <memory>
#include <experimental/optional> #include <experimental/optional>
#include <mapbox/variant.hpp>
#include <vector> #include <vector>
#include <clingo.hh>
namespace anthem namespace anthem
{ {
@ -64,34 +65,34 @@ using VariablePointer = std::unique_ptr<Variable>;
// Variants // Variants
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
using FormulaT = mapbox::util::variant< using FormulaT = Clingo::Variant<
AndPointer, And,
BiconditionalPointer, Biconditional,
BooleanPointer, Boolean,
ComparisonPointer, Comparison,
ExistsPointer, Exists,
ForAllPointer, ForAll,
ImpliesPointer, Implies,
InPointer, In,
NotPointer, Not,
OrPointer, Or,
PredicatePointer>; Predicate>;
class Formula : public FormulaT class Formula : public FormulaT
{ {
using FormulaT::FormulaT; using FormulaT::FormulaT;
}; };
using TermT = mapbox::util::variant< using TermT = Clingo::Variant<
BinaryOperationPointer, BinaryOperation,
BooleanPointer, Boolean,
ConstantPointer, Constant,
FunctionPointer, Function,
IntegerPointer, Integer,
IntervalPointer, Interval,
SpecialIntegerPointer, SpecialInteger,
StringPointer, String,
VariablePointer>; Variable>;
class Term : public TermT class Term : public TermT
{ {

View File

@ -40,11 +40,11 @@ ast::Comparison::Operator translate(Clingo::AST::ComparisonOperator comparisonOp
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
ast::VariablePointer makeAuxiliaryBodyVariable(const int i) ast::Variable makeAuxiliaryBodyVariable(const int i)
{ {
auto variableName = std::string(AuxiliaryBodyVariablePrefix) + std::to_string(i); auto variableName = std::string(AuxiliaryBodyVariablePrefix) + std::to_string(i);
return std::make_unique<ast::Variable>(std::move(variableName), ast::Variable::Type::Reserved); return ast::Variable(std::move(variableName), ast::Variable::Type::Reserved);
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
@ -88,36 +88,36 @@ struct BodyTermTranslateVisitor
throwErrorAtLocation(literal.location, "double-negated literals currently unsupported", context); throwErrorAtLocation(literal.location, "double-negated literals currently unsupported", context);
if (function.arguments.empty()) if (function.arguments.empty())
return std::make_unique<ast::Predicate>(std::string(function.name)); return ast::Formula::make<ast::Predicate>(std::string(function.name));
std::vector<ast::VariablePointer> variables; std::vector<ast::Variable> variables;
variables.reserve(function.arguments.size()); variables.reserve(function.arguments.size());
for (size_t i = 0; i < function.arguments.size(); i++) for (size_t i = 0; i < function.arguments.size(); i++)
variables.emplace_back(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID + i)); variables.emplace_back(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID + i));
auto conjunction = std::make_unique<ast::And>(); ast::And conjunction;
for (size_t i = 0; i < function.arguments.size(); i++) for (size_t i = 0; i < function.arguments.size(); i++)
{ {
const auto &argument = function.arguments[i]; const auto &argument = function.arguments[i];
conjunction->arguments.emplace_back(std::make_unique<ast::In>(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID + i), translate(argument, context))); conjunction.arguments.emplace_back(ast::Formula::make<ast::In>(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID + i), translate(argument, context)));
} }
auto predicate = std::make_unique<ast::Predicate>(std::string(function.name)); ast::Predicate predicate(std::string(function.name));
predicate->arguments.reserve(function.arguments.size()); predicate.arguments.reserve(function.arguments.size());
for (size_t i = 0; i < function.arguments.size(); i++) for (size_t i = 0; i < function.arguments.size(); i++)
predicate->arguments.emplace_back(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID + i)); predicate.arguments.emplace_back(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID + i));
if (literal.sign == Clingo::AST::Sign::None) if (literal.sign == Clingo::AST::Sign::None)
conjunction->arguments.emplace_back(std::move(predicate)); conjunction.arguments.emplace_back(std::move(predicate));
else else
conjunction->arguments.emplace_back(std::make_unique<ast::Not>(std::move(predicate))); conjunction.arguments.emplace_back(ast::Formula::make<ast::Not>(std::move(predicate)));
context.auxiliaryBodyLiteralID += function.arguments.size(); context.auxiliaryBodyLiteralID += function.arguments.size();
return std::make_unique<ast::Exists>(std::move(variables), std::move(conjunction)); return ast::Formula::make<ast::Exists>(std::move(variables), std::move(conjunction));
} }
std::experimental::optional<ast::Formula> visit(const Clingo::AST::Pool &, const Clingo::AST::Literal &, const Clingo::AST::Term &term, Context &context) std::experimental::optional<ast::Formula> visit(const Clingo::AST::Pool &, const Clingo::AST::Literal &, const Clingo::AST::Term &term, Context &context)
@ -150,20 +150,20 @@ struct BodyLiteralTranslateVisitor
const auto operator_ = translate(comparison.comparison); const auto operator_ = translate(comparison.comparison);
std::vector<ast::VariablePointer> variables; std::vector<ast::Variable> variables;
variables.reserve(2); variables.reserve(2);
variables.emplace_back(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID)); variables.emplace_back(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID));
variables.emplace_back(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID + 1)); variables.emplace_back(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID + 1));
auto conjunction = std::make_unique<ast::And>(); ast::And conjunction;
conjunction->arguments.reserve(3); conjunction.arguments.reserve(3);
conjunction->arguments.emplace_back(std::make_unique<ast::In>(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID), translate(comparison.left, context))); conjunction.arguments.emplace_back(ast::Formula::make<ast::In>(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID), translate(comparison.left, context)));
conjunction->arguments.emplace_back(std::make_unique<ast::In>(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID + 1), translate(comparison.right, context))); conjunction.arguments.emplace_back(ast::Formula::make<ast::In>(makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID + 1), translate(comparison.right, context)));
conjunction->arguments.emplace_back(std::make_unique<ast::Comparison>(operator_, makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID), makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID + 1))); conjunction.arguments.emplace_back(ast::Formula::make<ast::Comparison>(operator_, makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID), makeAuxiliaryBodyVariable(context.auxiliaryBodyLiteralID + 1)));
context.auxiliaryBodyLiteralID += 2; context.auxiliaryBodyLiteralID += 2;
return std::make_unique<ast::Exists>(std::move(variables), std::move(conjunction)); return ast::Formula::make<ast::Exists>(std::move(variables), std::move(conjunction));
} }
std::experimental::optional<ast::Formula> visit(const Clingo::AST::CSPLiteral &, const Clingo::AST::Literal &literal, Context &context) std::experimental::optional<ast::Formula> visit(const Clingo::AST::CSPLiteral &, const Clingo::AST::Literal &literal, Context &context)

View File

@ -188,10 +188,10 @@ struct FunctionTermTranslateVisitor
assert(matchingTerm != context.headTerms.cend()); assert(matchingTerm != context.headTerms.cend());
auto variableName = std::string(AuxiliaryHeadVariablePrefix) + std::to_string(matchingTerm - context.headTerms.cbegin() + 1); auto variableName = std::string(AuxiliaryHeadVariablePrefix) + std::to_string(matchingTerm - context.headTerms.cbegin() + 1);
arguments.emplace_back(std::make_unique<ast::Variable>(std::move(variableName), ast::Variable::Type::Reserved)); arguments.emplace_back(ast::Variable(std::move(variableName), ast::Variable::Type::Reserved));
} }
return std::make_unique<ast::Predicate>(function.name, std::move(arguments)); return ast::Formula::make<ast::Predicate>(function.name, std::move(arguments));
} }
std::experimental::optional<ast::Formula> visit(const Clingo::AST::Pool &, const Clingo::AST::Term &term, Context &context) std::experimental::optional<ast::Formula> visit(const Clingo::AST::Pool &, const Clingo::AST::Term &term, Context &context)
@ -207,7 +207,7 @@ struct LiteralTranslateVisitor
{ {
std::experimental::optional<ast::Formula> visit(const Clingo::AST::Boolean &boolean, const Clingo::AST::Literal &, Context &) std::experimental::optional<ast::Formula> visit(const Clingo::AST::Boolean &boolean, const Clingo::AST::Literal &, Context &)
{ {
return std::make_unique<ast::Boolean>(boolean.value); return ast::Formula::make<ast::Boolean>(boolean.value);
} }
std::experimental::optional<ast::Formula> visit(const Clingo::AST::Term &term, const Clingo::AST::Literal &, Context &context) std::experimental::optional<ast::Formula> visit(const Clingo::AST::Term &term, const Clingo::AST::Literal &, Context &context)
@ -245,7 +245,7 @@ struct HeadLiteralTranslateToConsequentVisitor
if (!translatedLiteral) if (!translatedLiteral)
return std::experimental::nullopt; return std::experimental::nullopt;
return std::make_unique<ast::Not>(std::move(translatedLiteral.value())); return ast::Formula::make<ast::Not>(std::move(translatedLiteral.value()));
} }
std::experimental::optional<ast::Formula> visit(const Clingo::AST::Disjunction &disjunction, const Clingo::AST::HeadLiteral &headLiteral, Context &context) std::experimental::optional<ast::Formula> visit(const Clingo::AST::Disjunction &disjunction, const Clingo::AST::HeadLiteral &headLiteral, Context &context)
@ -266,7 +266,7 @@ struct HeadLiteralTranslateToConsequentVisitor
arguments.emplace_back(std::move(argument.value())); arguments.emplace_back(std::move(argument.value()));
} }
return std::make_unique<ast::Or>(std::move(arguments)); return ast::Formula::make<ast::Or>(std::move(arguments));
} }
std::experimental::optional<ast::Formula> visit(const Clingo::AST::Aggregate &aggregate, const Clingo::AST::HeadLiteral &headLiteral, Context &context) std::experimental::optional<ast::Formula> visit(const Clingo::AST::Aggregate &aggregate, const Clingo::AST::HeadLiteral &headLiteral, Context &context)
@ -299,7 +299,7 @@ struct HeadLiteralTranslateToConsequentVisitor
arguments.emplace_back(std::move(argument.value())); arguments.emplace_back(std::move(argument.value()));
} }
return std::make_unique<ast::Or>(std::move(arguments)); return ast::Formula::make<ast::Or>(std::move(arguments));
} }
std::experimental::optional<ast::Formula> visit(const Clingo::AST::HeadAggregate &, const Clingo::AST::HeadLiteral &headLiteral, Context &context) std::experimental::optional<ast::Formula> visit(const Clingo::AST::HeadAggregate &, const Clingo::AST::HeadLiteral &headLiteral, Context &context)

View File

@ -12,7 +12,7 @@ namespace anthem
// //
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
ast::Formula simplify(ast::Formula &&formula); void simplify(ast::Formula &formula);
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////

View File

@ -36,7 +36,7 @@ struct StatementVisitor
// Concatenate all head terms // Concatenate all head terms
rule.head.data.accept(HeadLiteralCollectFunctionTermsVisitor(), rule.head, context); rule.head.data.accept(HeadLiteralCollectFunctionTermsVisitor(), rule.head, context);
auto antecedent = std::make_unique<ast::And>(); ast::And antecedent;
// Compute consequent // Compute consequent
auto consequent = rule.head.data.accept(HeadLiteralTranslateToConsequentVisitor(), rule.head, context); auto consequent = rule.head.data.accept(HeadLiteralTranslateToConsequentVisitor(), rule.head, context);
@ -53,11 +53,11 @@ struct StatementVisitor
const auto &headTerm = **i; const auto &headTerm = **i;
auto variableName = std::string(AuxiliaryHeadVariablePrefix) + std::to_string(i - context.headTerms.cbegin() + 1); auto variableName = std::string(AuxiliaryHeadVariablePrefix) + std::to_string(i - context.headTerms.cbegin() + 1);
auto element = std::make_unique<ast::Variable>(std::move(variableName), ast::Variable::Type::Reserved); auto element = ast::Variable(std::move(variableName), ast::Variable::Type::Reserved);
auto set = translate(headTerm, context); auto set = translate(headTerm, context);
auto in = std::make_unique<ast::In>(std::move(element), std::move(set)); auto in = ast::In(std::move(element), std::move(set));
antecedent->arguments.emplace_back(std::move(in)); antecedent.arguments.emplace_back(std::move(in));
} }
// Print translated body literals // Print translated body literals
@ -73,20 +73,20 @@ struct StatementVisitor
if (!argument) if (!argument)
throwErrorAtLocation(bodyLiteral.location, "could not translate body literal", context); throwErrorAtLocation(bodyLiteral.location, "could not translate body literal", context);
antecedent->arguments.emplace_back(std::move(argument.value())); antecedent.arguments.emplace_back(std::move(argument.value()));
} }
// Handle choice rules // Handle choice rules
if (context.isChoiceRule) if (context.isChoiceRule)
antecedent->arguments.emplace_back(ast::deepCopy(consequent.value())); antecedent.arguments.emplace_back(ast::deepCopy(consequent.value()));
// Use “true” as the consequent in case it is empty // Use “true” as the consequent in case it is empty
if (antecedent->arguments.empty()) if (antecedent.arguments.empty())
return std::make_unique<ast::Implies>(std::make_unique<ast::Boolean>(true), std::move(consequent.value())); return ast::Formula::make<ast::Implies>(ast::Boolean(true), std::move(consequent.value()));
else if (antecedent->arguments.size() == 1) else if (antecedent.arguments.size() == 1)
return std::make_unique<ast::Implies>(std::move(antecedent->arguments[0]), std::move(consequent.value())); return ast::Formula::make<ast::Implies>(std::move(antecedent.arguments[0]), std::move(consequent.value()));
return std::make_unique<ast::Implies>(std::move(antecedent), std::move(consequent.value())); return ast::Formula::make<ast::Implies>(std::move(antecedent), std::move(consequent.value()));
} }
std::experimental::optional<ast::Formula> visit(const Clingo::AST::Definition &, const Clingo::AST::Statement &statement, Context &context) std::experimental::optional<ast::Formula> visit(const Clingo::AST::Definition &, const Clingo::AST::Statement &statement, Context &context)

View File

@ -50,17 +50,19 @@ struct TermTranslateVisitor
switch (symbol.type()) switch (symbol.type())
{ {
case Clingo::SymbolType::Number: case Clingo::SymbolType::Number:
return std::make_unique<ast::Integer>(symbol.number()); return ast::Term::make<ast::Integer>(symbol.number());
case Clingo::SymbolType::Infimum: case Clingo::SymbolType::Infimum:
return std::make_unique<ast::SpecialInteger>(ast::SpecialInteger::Type::Infimum); return ast::Term::make<ast::SpecialInteger>(ast::SpecialInteger::Type::Infimum);
case Clingo::SymbolType::Supremum: case Clingo::SymbolType::Supremum:
return std::make_unique<ast::SpecialInteger>(ast::SpecialInteger::Type::Supremum); return ast::Term::make<ast::SpecialInteger>(ast::SpecialInteger::Type::Supremum);
case Clingo::SymbolType::String: case Clingo::SymbolType::String:
return std::make_unique<ast::String>(std::string(symbol.string())); return ast::Term::make<ast::String>(std::string(symbol.string()));
case Clingo::SymbolType::Function: case Clingo::SymbolType::Function:
{ {
auto function = std::make_unique<ast::Function>(symbol.name()); auto function = ast::Term::make<ast::Function>(symbol.name());
function->arguments.reserve(symbol.arguments().size()); // TODO: remove workaround
auto &functionRaw = function.get<ast::Function>();
functionRaw.arguments.reserve(symbol.arguments().size());
for (const auto &argument : symbol.arguments()) for (const auto &argument : symbol.arguments())
{ {
@ -69,7 +71,7 @@ struct TermTranslateVisitor
if (!translatedArgument) if (!translatedArgument)
throwErrorAtLocation(term.location, "could not translate argument", context); throwErrorAtLocation(term.location, "could not translate argument", context);
function->arguments.emplace_back(std::move(translatedArgument.value())); functionRaw.arguments.emplace_back(std::move(translatedArgument.value()));
} }
return std::move(function); return std::move(function);
@ -83,7 +85,7 @@ struct TermTranslateVisitor
std::experimental::optional<ast::Term> visit(const Clingo::AST::Variable &variable, const Clingo::AST::Term &, Context &) std::experimental::optional<ast::Term> visit(const Clingo::AST::Variable &variable, const Clingo::AST::Term &, Context &)
{ {
return std::make_unique<ast::Variable>(std::string(variable.name), ast::Variable::Type::UserDefined); return ast::Term::make<ast::Variable>(std::string(variable.name), ast::Variable::Type::UserDefined);
} }
std::experimental::optional<ast::Term> visit(const Clingo::AST::UnaryOperation &, const Clingo::AST::Term &term, Context &context) std::experimental::optional<ast::Term> visit(const Clingo::AST::UnaryOperation &, const Clingo::AST::Term &term, Context &context)
@ -98,7 +100,7 @@ struct TermTranslateVisitor
auto left = translate(binaryOperation.left, context); auto left = translate(binaryOperation.left, context);
auto right = translate(binaryOperation.right, context); auto right = translate(binaryOperation.right, context);
return std::make_unique<ast::BinaryOperation>(operator_, std::move(left), std::move(right)); return ast::Term::make<ast::BinaryOperation>(operator_, std::move(left), std::move(right));
} }
std::experimental::optional<ast::Term> visit(const Clingo::AST::Interval &interval, const Clingo::AST::Term &, Context &context) std::experimental::optional<ast::Term> visit(const Clingo::AST::Interval &interval, const Clingo::AST::Term &, Context &context)
@ -106,7 +108,7 @@ struct TermTranslateVisitor
auto left = translate(interval.left, context); auto left = translate(interval.left, context);
auto right = translate(interval.right, context); auto right = translate(interval.right, context);
return std::make_unique<ast::Interval>(std::move(left), std::move(right)); return ast::Term::make<ast::Interval>(std::move(left), std::move(right));
} }
std::experimental::optional<ast::Term> visit(const Clingo::AST::Function &function, const Clingo::AST::Term &term, Context &context) std::experimental::optional<ast::Term> visit(const Clingo::AST::Function &function, const Clingo::AST::Term &term, Context &context)
@ -120,7 +122,7 @@ struct TermTranslateVisitor
for (const auto &argument : function.arguments) for (const auto &argument : function.arguments)
arguments.emplace_back(translate(argument, context)); arguments.emplace_back(translate(argument, context));
return std::make_unique<ast::Function>(function.name, std::move(arguments)); return ast::Term::make<ast::Function>(function.name, std::move(arguments));
} }
std::experimental::optional<ast::Term> visit(const Clingo::AST::Pool &, const Clingo::AST::Term &term, Context &context) std::experimental::optional<ast::Term> visit(const Clingo::AST::Pool &, const Clingo::AST::Term &term, Context &context)

View File

@ -48,7 +48,7 @@ output::ColorStream &operator<<(output::ColorStream &stream, const Term &term);
// Primitives // Primitives
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, BinaryOperation::Operator operator_) inline output::ColorStream &operator<<(output::ColorStream &stream, BinaryOperation::Operator operator_)
{ {
switch (operator_) switch (operator_)
{ {
@ -69,14 +69,14 @@ output::ColorStream &operator<<(output::ColorStream &stream, BinaryOperation::Op
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const BinaryOperation &binaryOperation) inline output::ColorStream &operator<<(output::ColorStream &stream, const BinaryOperation &binaryOperation)
{ {
return (stream << "(" << binaryOperation.left << " " << binaryOperation.operator_ << " " << binaryOperation.right << ")"); return (stream << "(" << binaryOperation.left << " " << binaryOperation.operator_ << " " << binaryOperation.right << ")");
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Boolean &boolean) inline output::ColorStream &operator<<(output::ColorStream &stream, const Boolean &boolean)
{ {
if (boolean.value) if (boolean.value)
return (stream << output::Boolean("#true")); return (stream << output::Boolean("#true"));
@ -86,7 +86,7 @@ output::ColorStream &operator<<(output::ColorStream &stream, const Boolean &bool
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, Comparison::Operator operator_) inline output::ColorStream &operator<<(output::ColorStream &stream, Comparison::Operator operator_)
{ {
switch (operator_) switch (operator_)
{ {
@ -109,21 +109,21 @@ output::ColorStream &operator<<(output::ColorStream &stream, Comparison::Operato
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Comparison &comparison) inline output::ColorStream &operator<<(output::ColorStream &stream, const Comparison &comparison)
{ {
return (stream << comparison.left << " " << comparison.operator_ << " " << comparison.right); return (stream << comparison.left << " " << comparison.operator_ << " " << comparison.right);
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Constant &constant) inline output::ColorStream &operator<<(output::ColorStream &stream, const Constant &constant)
{ {
return (stream << constant.name); return (stream << constant.name);
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Function &function) inline output::ColorStream &operator<<(output::ColorStream &stream, const Function &function)
{ {
stream << function.name; stream << function.name;
@ -148,28 +148,28 @@ output::ColorStream &operator<<(output::ColorStream &stream, const Function &fun
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const In &in) inline output::ColorStream &operator<<(output::ColorStream &stream, const In &in)
{ {
return (stream << in.element << " " << output::Keyword("in") << " " << in.set); return (stream << in.element << " " << output::Keyword("in") << " " << in.set);
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Integer &integer) inline output::ColorStream &operator<<(output::ColorStream &stream, const Integer &integer)
{ {
return (stream << output::Number<int>(integer.value)); return (stream << output::Number<int>(integer.value));
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Interval &interval) inline output::ColorStream &operator<<(output::ColorStream &stream, const Interval &interval)
{ {
return (stream << interval.from << ".." << interval.to); return (stream << interval.from << ".." << interval.to);
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Predicate &predicate) inline output::ColorStream &operator<<(output::ColorStream &stream, const Predicate &predicate)
{ {
stream << predicate.name; stream << predicate.name;
@ -191,7 +191,7 @@ output::ColorStream &operator<<(output::ColorStream &stream, const Predicate &pr
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const SpecialInteger &specialInteger) inline output::ColorStream &operator<<(output::ColorStream &stream, const SpecialInteger &specialInteger)
{ {
switch (specialInteger.type) switch (specialInteger.type)
{ {
@ -206,14 +206,14 @@ output::ColorStream &operator<<(output::ColorStream &stream, const SpecialIntege
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const String &string) inline output::ColorStream &operator<<(output::ColorStream &stream, const String &string)
{ {
return (stream << output::String(string.text.c_str())); return (stream << output::String(string.text.c_str()));
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Variable &variable) inline output::ColorStream &operator<<(output::ColorStream &stream, const Variable &variable)
{ {
assert(!variable.name.empty()); assert(!variable.name.empty());
assert(variable.name[0] >= 65 && variable.name[0] <= 90); assert(variable.name[0] >= 65 && variable.name[0] <= 90);
@ -230,7 +230,7 @@ output::ColorStream &operator<<(output::ColorStream &stream, const Variable &var
// Expressions // Expressions
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const And &and_) inline output::ColorStream &operator<<(output::ColorStream &stream, const And &and_)
{ {
stream << "("; stream << "(";
@ -247,14 +247,14 @@ output::ColorStream &operator<<(output::ColorStream &stream, const And &and_)
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Biconditional &biconditional) inline output::ColorStream &operator<<(output::ColorStream &stream, const Biconditional &biconditional)
{ {
return (stream << "(" << biconditional.left << " <-> " << biconditional.right << ")"); return (stream << "(" << biconditional.left << " <-> " << biconditional.right << ")");
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Exists &exists) inline output::ColorStream &operator<<(output::ColorStream &stream, const Exists &exists)
{ {
stream << output::Keyword("exists") << " "; stream << output::Keyword("exists") << " ";
@ -263,7 +263,7 @@ output::ColorStream &operator<<(output::ColorStream &stream, const Exists &exist
if (i != exists.variables.cbegin()) if (i != exists.variables.cbegin())
stream << ", "; stream << ", ";
stream << (**i); stream << (*i);
} }
return (stream << " " << exists.argument); return (stream << " " << exists.argument);
@ -271,7 +271,7 @@ output::ColorStream &operator<<(output::ColorStream &stream, const Exists &exist
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const ForAll &forAll) inline output::ColorStream &operator<<(output::ColorStream &stream, const ForAll &forAll)
{ {
stream << output::Keyword("forall") << " "; stream << output::Keyword("forall") << " ";
@ -280,7 +280,7 @@ output::ColorStream &operator<<(output::ColorStream &stream, const ForAll &forAl
if (i != forAll.variables.cbegin()) if (i != forAll.variables.cbegin())
stream << ", "; stream << ", ";
stream << (**i); stream << (*i);
} }
return (stream << " " << forAll.argument); return (stream << " " << forAll.argument);
@ -288,21 +288,21 @@ output::ColorStream &operator<<(output::ColorStream &stream, const ForAll &forAl
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Implies &implies) inline output::ColorStream &operator<<(output::ColorStream &stream, const Implies &implies)
{ {
return (stream << "(" << implies.antecedent << " -> " << implies.consequent << ")"); return (stream << "(" << implies.antecedent << " -> " << implies.consequent << ")");
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Not &not_) inline output::ColorStream &operator<<(output::ColorStream &stream, const Not &not_)
{ {
return (stream << output::Keyword("not ") << not_.argument); return (stream << output::Keyword("not ") << not_.argument);
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Or &or_) inline output::ColorStream &operator<<(output::ColorStream &stream, const Or &or_)
{ {
stream << "("; stream << "(";
@ -321,20 +321,28 @@ output::ColorStream &operator<<(output::ColorStream &stream, const Or &or_)
// Variants // Variants
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Formula &formula) template<class Variant>
struct VariantPrintVisitor
{ {
formula.match([&](const auto &x){stream << *x;}); template<class T>
output::ColorStream &visit(const T &x, output::ColorStream &stream)
{
return (stream << x);
}
};
return stream; ////////////////////////////////////////////////////////////////////////////////////////////////////
inline output::ColorStream &operator<<(output::ColorStream &stream, const Formula &formula)
{
return formula.accept(VariantPrintVisitor<ast::Formula>(), stream);
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
output::ColorStream &operator<<(output::ColorStream &stream, const Term &term) inline output::ColorStream &operator<<(output::ColorStream &stream, const Term &term)
{ {
term.match([&](const auto &x){stream << *x;}); return term.accept(VariantPrintVisitor<ast::Term>(), stream);
return stream;
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////

@ -1 +0,0 @@
Subproject commit d2588a8f1d6b5d480d228e6d8a906ce634bdea9a

View File

@ -1,7 +1,5 @@
#include <anthem/Simplification.h> #include <anthem/Simplification.h>
#include <iostream>
namespace anthem namespace anthem
{ {
@ -13,92 +11,139 @@ namespace anthem
bool isPrimitiveTerm(const ast::Term &term) bool isPrimitiveTerm(const ast::Term &term)
{ {
const auto handleBinaryOperation = return (!term.is<ast::BinaryOperation>() && !term.is<ast::Interval>());
[](const ast::BinaryOperationPointer &)
{
return false;
};
const auto handleInterval =
[](const ast::IntervalPointer &)
{
return false;
};
const auto handleDefault =
[](const auto &)
{
return true;
};
return term.match(handleBinaryOperation, handleInterval, handleDefault);
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
bool matchesVariable(const ast::Term &term, const ast::VariablePointer &variable) template<class T>
struct RecursiveFormulaVisitor
{ {
const auto handleVariable = void visit(ast::And &and_, ast::Formula &formula)
[&](const ast::VariablePointer &other) {
{ for (auto &argument : and_.arguments)
return variable->name == other->name; argument.accept(*this, argument);
};
const auto handleDefault = return T::accept(and_, formula);
[](const auto &) }
{
return false;
};
return term.match(handleVariable, handleDefault); void visit(ast::Biconditional &biconditional, ast::Formula &formula)
{
biconditional.left.accept(*this, biconditional.left);
biconditional.right.accept(*this, biconditional.right);
return T::accept(biconditional, formula);
}
void visit(ast::Boolean &boolean, ast::Formula &formula)
{
return T::accept(boolean, formula);
}
void visit(ast::Comparison &comparison, ast::Formula &formula)
{
return T::accept(comparison, formula);
}
void visit(ast::Exists &exists, ast::Formula &formula)
{
exists.argument.accept(*this, exists.argument);
return T::accept(exists, formula);
}
void visit(ast::ForAll &forAll, ast::Formula &formula)
{
forAll.argument.accept(*this, forAll.argument);
return T::accept(forAll, formula);
}
void visit(ast::Implies &implies, ast::Formula &formula)
{
implies.antecedent.accept(*this, implies.antecedent);
implies.consequent.accept(*this, implies.consequent);
return T::accept(implies, formula);
}
void visit(ast::In &in, ast::Formula &formula)
{
return T::accept(in, formula);
}
void visit(ast::Not &not_, ast::Formula &formula)
{
not_.argument.accept(*this, not_.argument);
return T::accept(not_, formula);
}
void visit(ast::Or &or_, ast::Formula &formula)
{
for (auto &argument : or_.arguments)
argument.accept(*this, argument);
return T::accept(or_, formula);
}
void visit(ast::Predicate &predicate, ast::Formula &formula)
{
return T::accept(predicate, formula);
}
};
////////////////////////////////////////////////////////////////////////////////////////////////////
bool matchesVariable(const ast::Term &term, const ast::Variable &variable)
{
if (!term.is<ast::Variable>())
return false;
const auto otherVariable = term.get<ast::Variable>();
return variable.name == otherVariable.name;
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
std::experimental::optional<ast::Term> extractAssignedTerm(ast::Formula &formula, const ast::VariablePointer &variable) std::experimental::optional<ast::Term> extractAssignedTerm(ast::Formula &formula, const ast::Variable &variable)
{ {
const auto handleComparison = if (!formula.is<ast::Comparison>())
[&](ast::ComparisonPointer &comparison) -> std::experimental::optional<ast::Term> return std::experimental::nullopt;
{
if (comparison->operator_ != ast::Comparison::Operator::Equal)
return std::experimental::nullopt;
if (matchesVariable(comparison->left, variable)) auto &comparison = formula.get<ast::Comparison>();
return std::move(comparison->right);
if (matchesVariable(comparison->right, variable)) if (comparison.operator_ != ast::Comparison::Operator::Equal)
return std::move(comparison->left); return std::experimental::nullopt;
return std::experimental::nullopt; if (matchesVariable(comparison.left, variable))
}; return std::move(comparison.right);
const auto handleDefault = if (matchesVariable(comparison.right, variable))
[](auto &) -> std::experimental::optional<ast::Term> return std::move(comparison.left);
{
return std::experimental::nullopt;
};
return formula.match(handleComparison, handleDefault); return std::experimental::nullopt;
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
ast::Formula simplify(ast::ExistsPointer &exists) void simplify(ast::Exists &exists, ast::Formula &formula)
{ {
if (!exists->argument.is<ast::AndPointer>()) if (!exists.argument.is<ast::And>())
return std::move(exists); return;
auto &conjunction = exists->argument.get_unchecked<ast::AndPointer>(); auto &conjunction = exists.argument.get<ast::And>();
auto &arguments = conjunction->arguments; auto &arguments = conjunction.arguments;
// Check that formula is in normal form // Check that formula is in normal form
if (!arguments.back().is<ast::PredicatePointer>()) if (!arguments.back().is<ast::Predicate>())
return std::move(exists); return;
const auto replaceVariableInPredicateWithTerm = const auto replaceVariableInPredicateWithTerm =
[](ast::PredicatePointer &predicate, const ast::VariablePointer &variable, ast::Term &&term) [](ast::Predicate &predicate, const ast::Variable &variable, ast::Term &term)
{ {
for (auto &argument : predicate->arguments) for (auto &argument : predicate.arguments)
{ {
if (!matchesVariable(argument, variable)) if (!matchesVariable(argument, variable))
continue; continue;
@ -109,7 +154,7 @@ ast::Formula simplify(ast::ExistsPointer &exists)
}; };
// Simplify formulas of type “exists X (X = t and F(Y))” to “F(t)” // Simplify formulas of type “exists X (X = t and F(Y))” to “F(t)”
for (auto i = exists->variables.begin(); i != exists->variables.end();) for (auto i = exists.variables.begin(); i != exists.variables.end();)
{ {
auto &variable = *i; auto &variable = *i;
@ -123,10 +168,10 @@ ast::Formula simplify(ast::ExistsPointer &exists)
if (!assignedTerm) if (!assignedTerm)
continue; continue;
auto &lastArgument = arguments.back().get_unchecked<ast::PredicatePointer>(); auto &lastArgument = arguments.back().get<ast::Predicate>();
// If this argument is an assignment of the variable to some other term, remove the assignment and replace the variable with the other term // If this argument is an assignment of the variable to some other term, remove the assignment and replace the variable with the other term
replaceVariableInPredicateWithTerm(lastArgument, variable, std::move(assignedTerm.value())); replaceVariableInPredicateWithTerm(lastArgument, variable, assignedTerm.value());
arguments.erase(j); arguments.erase(j);
wasVariableReplaced = true; wasVariableReplaced = true;
@ -135,107 +180,60 @@ ast::Formula simplify(ast::ExistsPointer &exists)
if (wasVariableReplaced) if (wasVariableReplaced)
{ {
i = exists->variables.erase(i); i = exists.variables.erase(i);
continue; continue;
} }
i++; i++;
} }
if (exists->variables.empty()) // If there are still variables, do nothing more
if (!exists.variables.empty())
return;
assert(!conjunction.arguments.empty());
// If the argument is a conjunction with just one element, directly replace the input formula with the argument
if (conjunction.arguments.size() == 1)
{ {
assert(!conjunction->arguments.empty()); auto test = std::move(conjunction.arguments.front());
formula = std::move(test);
if (conjunction->arguments.size() == 1) return;
return std::move(conjunction->arguments.front());
return std::move(exists->argument);
} }
return std::move(exists); // If there is more than one element in the conjunction, replace the input formula with the conjunction
formula = std::move(exists.argument);
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////
ast::Formula simplify(ast::Formula &&formula) struct SimplifyFormulaVisitor : public RecursiveFormulaVisitor<SimplifyFormulaVisitor>
{ {
const auto handleAnd = static void accept(ast::Exists &exists, ast::Formula &formula)
[&](ast::AndPointer &and_) -> ast::Formula {
{ simplify(exists, formula);
for (auto &argument : and_->arguments) }
argument = simplify(std::move(argument));
return std::move(and_); static void accept(ast::In &in, ast::Formula &formula)
}; {
if (!isPrimitiveTerm(in.element) || !isPrimitiveTerm(in.set))
return;
const auto handleBiconditional = // Simplify formulas of type “A in B” to “A = B” if A and B are primitive
[&](ast::BiconditionalPointer &biconditional) -> ast::Formula formula = ast::Comparison(ast::Comparison::Operator::Equal, std::move(in.element), std::move(in.set));
{ }
biconditional->left = simplify(std::move(biconditional->left));
biconditional->right = simplify(std::move(biconditional->right));
return std::move(biconditional); template<class T>
}; static void accept(T &, ast::Formula &)
{
}
};
const auto handleExists = ////////////////////////////////////////////////////////////////////////////////////////////////////
[&](ast::ExistsPointer &exists) -> ast::Formula
{
exists->argument = simplify(std::move(exists->argument));
return simplify(exists); void simplify(ast::Formula &formula)
}; {
formula.accept(SimplifyFormulaVisitor(), formula);
const auto handleForAll =
[&](ast::ForAllPointer &forAll) -> ast::Formula
{
forAll->argument = simplify(std::move(forAll->argument));
return std::move(forAll);
};
const auto handleImplies =
[&](ast::ImpliesPointer &implies) -> ast::Formula
{
implies->antecedent = simplify(std::move(implies->antecedent));
implies->consequent = simplify(std::move(implies->consequent));
return std::move(implies);
};
const auto handleIn =
[](ast::InPointer &in) -> ast::Formula
{
if (!isPrimitiveTerm(in->element) || !isPrimitiveTerm(in->set))
return std::move(in);
// Simplify formulas of type “A in B” to “A = B” if A and B are primitive
return std::make_unique<ast::Comparison>(ast::Comparison::Operator::Equal, std::move(in->element), std::move(in->set));
};
const auto handleNot =
[&](ast::NotPointer &not_) -> ast::Formula
{
not_->argument = simplify(std::move(not_->argument));
return std::move(not_);
};
const auto handleOr =
[&](ast::OrPointer &or_) -> ast::Formula
{
for (auto &argument : or_->arguments)
argument = simplify(std::move(argument));
return std::move(or_);
};
const auto handleDefault =
[&](auto &x) -> ast::Formula
{
return std::move(x);
};
return formula.match(handleAnd, handleBiconditional, handleExists, handleForAll, handleImplies, handleIn, handleNot, handleOr, handleDefault);
} }
//////////////////////////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////////////////////////

View File

@ -53,8 +53,8 @@ void translate(const char *fileName, std::istream &stream, Context &context)
return; return;
} }
auto simplifiedFormula = simplify(std::move(formula.value())); simplify(formula.value());
context.logger.outputStream() << simplifiedFormula << std::endl; context.logger.outputStream() << formula.value() << std::endl;
}; };
const auto logger = const auto logger =