fix: merge_strategy() is an instance, not a type (#164)

* merge_strategy is an instance not a type

* unused type parameters: silence warnings

* bump to 0.8.7

Author: kalmarek

Project.toml

@@ -1,7 +1,7 @@
 name = "COSMO"
 uuid = "1e616198-aa4e-51ec-90a2-23f7fbd31d8d"
 authors = ["Michael <[email protected]>"]
-version = "0.8.6"
+version = "0.8.7"
 
 [deps]
 AMD = "14f7f29c-3bd6-536c-9a0b-7339e30b5a3e"

src/accelerator_interface.jl

@@ -35,7 +35,7 @@ end
 check_activation!(ws::Workspace, activation_reason::AccuracyActivation, num_iter::Int64) = nothing
 
 
-function check_activation!(ws::Workspace, activation_reason::AccuracyActivation, r::ResultInfo) where {T <: AbstractFloat}
+function check_activation!(ws::Workspace, activation_reason::AccuracyActivation, r::ResultInfo)
     if !ws.accelerator_active
         tol = activation_reason.start_accuracy
 
@@ -45,7 +45,7 @@ function check_activation!(ws::Workspace, activation_reason::AccuracyActivation,
     end
 end
 
-check_activation!(ws::Workspace, activation_reason::Union{IterActivation, ImmediateActivation}, r::ResultInfo) where {T <: AbstractFloat}= nothing
+check_activation!(ws::Workspace, activation_reason::Union{IterActivation, ImmediateActivation}, r::ResultInfo) = nothing
 
 
 """

src/types.jl

@@ -197,7 +197,7 @@ mutable struct SparsityPattern
     # clique merging only if more than one clique present
     if sntree.num > 1
 		merge_cliques!(sntree)
-	elseif merge_strategy <: AbstractGraphBasedMerge
+	elseif merge_strategy isa AbstractGraphBasedMerge
 		# if no merging attempt happens for a clique graph, we still have to convert the snd and sep back to Array{Array{Int}, 1}
 		# for consistency
 		sntree.snd = sort.(collect.(sntree.snd))