Updating tags to set, needs more testing

Author: GearsAD

src/entities/DFGFactor.jl

@@ -47,7 +47,7 @@ mutable struct DFGFactor{T, S} <: AbstractDFGFactor
     timestamp::DateTime
     """Factor tags, e.g [:FACTOR].
     Accessors: `getTags`, `addTags!`, and `deleteTags!`"""
-    tags::Vector{Symbol}
+    tags::Set{Symbol}
     """Solver data.
     Accessors: `getSolverData`, `setSolverData!`"""
     solverData::GenericFunctionNodeData{T, S}

src/entities/DFGFactorSummary.jl

@@ -14,8 +14,8 @@ struct DFGFactorSummary <: AbstractDFGFactor
     Accessors: `getTimestamp`"""
     timestamp::DateTime
     """Factor tags, e.g [:FACTOR].
-    Accessors: `getTags`"""
-    tags::Vector{Symbol}
+    Accessors: `getTags`, `addTags!`, and `deleteTags!`"""
+    tags::Set{Symbol}
     """Internal ID used by some of the DFG drivers. We don't suggest using this outside of DFG."""
     _internalId::Int64
     """Internal cache of the ordering of the neighbor variables. Rather use getNeighbors to get the list as this is an internal value.

src/entities/DFGVariableSummary.jl

@@ -15,7 +15,7 @@ struct DFGVariableSummary <: AbstractDFGVariable
     timestamp::DateTime
     """Variable tags, e.g [:POSE, :VARIABLE, and :LANDMARK].
     Accessors: `getTags`, `addTags!`, and `deleteTags!`"""
-    tags::Vector{Symbol}
+    tags::Set{Symbol}
     """Dictionary of estimates keyed by solverDataDict keys
     Accessors: `addEstimate!`, `updateEstimate!`, and `deleteEstimate!`"""
     estimateDict::Dict{Symbol, <:AbstractPointParametricEst}

src/entities/SkeletonDFGFactor.jl

@@ -11,8 +11,8 @@ struct SkeletonDFGFactor <: AbstractDFGFactor
     Accessor: `getLabel`"""
     label::Symbol
     """Factor tags, e.g [:FACTOR].
-    Accessors: `getLabels`, `addLabels!`, and `deleteLabels!`"""
-    tags::Vector{Symbol}
+    Accessors: `getTags`, `addTags!`, and `deleteTags!`"""
+    tags::Set{Symbol}
     """Internal cache of the ordering of the neighbor variables. Rather use getNeighbors to get the list as this is an internal value.
     Accessors: `getVariableOrder`"""
     _variableOrderSymbols::Vector{Symbol}

src/entities/SkeletonDFGVariable.jl

@@ -12,8 +12,8 @@ struct SkeletonDFGVariable <: AbstractDFGVariable
     Accessor: `getLabel`"""
     label::Symbol
     """Variable tags, e.g [:POSE, :VARIABLE, and :LANDMARK].
-    Accessors: `getLabels`, `addLabels!`, and `deleteLabels!`"""
-    tags::Vector{Symbol}
+    Accessors: `getTags`, `addTags!`, and `deleteTags!`"""
+    tags::Set{Symbol}
 end
 
 SkeletonDFGVariable(label::Symbol) = SkeletonDFGVariable(label, Symbol[])