Skip to content

Commit

Permalink
Try to sort out the homs
Browse files Browse the repository at this point in the history
  • Loading branch information
lgoettgens committed May 3, 2024
1 parent 1f2ae64 commit fdf0b25
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
1 change: 1 addition & 0 deletions src/exports.jl
Original file line number Diff line number Diff line change
Expand Up @@ -241,6 +241,7 @@ export hnf_via_popov
export hnf_via_popov_with_transform
export hnf_with_transform
export hom
export hom_direct_sum
export hooklength
export ideal
export identity_map
Expand Down
4 changes: 2 additions & 2 deletions src/generic/DirectSum.jl
Original file line number Diff line number Diff line change
Expand Up @@ -252,12 +252,12 @@ function ModuleHomomorphism(D::DirectSumModule{T}, A::DirectSumModule{T}, m::Mat
return ModuleHomomorphism(D, A, transpose(hvcat(Tuple([length(SD) for i = 1:length(SA)]), map(x->transpose(x.matrix), m)...)))
end

function hom(M::DirectSumModule{T}, N::DirectSumModule{T}, mp::Vector{ModuleHomomorphism{T}}) where T
function hom_direct_sum(M::DirectSumModule{T}, N::DirectSumModule{T}, mp::Vector{ModuleHomomorphism{T}}) where T

Check warning on line 255 in src/generic/DirectSum.jl

View check run for this annotation

Codecov / codecov/patch

src/generic/DirectSum.jl#L255

Added line #L255 was not covered by tests
@assert length(M.m) == length(mp) == length(N.m)
return hom(M, N, cat(map(matrix, mp)..., dims = (1,2)))
end

function hom(A::DirectSumModule{T}, B::DirectSumModule{T}, M::Matrix{<:Map{<:AbstractAlgebra.FPModule{T}, <:AbstractAlgebra.FPModule{T}}}) where {T}
function hom_direct_sum(A::DirectSumModule{T}, B::DirectSumModule{T}, M::Matrix{<:Map{<:AbstractAlgebra.FPModule{T}, <:AbstractAlgebra.FPModule{T}}}) where {T}

Check warning on line 260 in src/generic/DirectSum.jl

View check run for this annotation

Codecov / codecov/patch

src/generic/DirectSum.jl#L260

Added line #L260 was not covered by tests
pro = canonical_projections(A)
im = canonical_injections(B)
s = hom(A, B, [zero(B) for i = 1:dim(A)])
Expand Down
1 change: 1 addition & 0 deletions src/generic/exports.jl
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ export groebner_basis
export has_bottom_neighbor
export has_left_neighbor
export hash
export hom_direct_sum
export hooklength
export image_fn
export image_map
Expand Down

0 comments on commit fdf0b25

Please # to comment.