Hi, I have wrapper code available for using CCOLAMD -- it's probably best to consolida

<a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/us

Hi <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="

Thanks for your answer <a class="user-mention notranslate" data-hovercard-type="user"

Consolidate CCOLAMD wrapper from IncrementalInference.jl?,about juliasmoothoptimizers/amd.jl

Comments (5)

dehann commented on June 9, 2024

Just to clarify on license -- we have strong reasoning for keeping permissive licenses throughout specific slices of the code base. Just copying here from JuliaRobotics/IncrementalInference.jl#980 (comment)

One solution might be to have JuliaRobotics/VariableEliminationOrderings.jl provides a mechanism to include more methods depending on the desired license restrictions.

from amd.jl.

dpo commented on June 9, 2024

Thank you. We're open to relicensing this package under the MIT license. Would that work for you?

from amd.jl.

amontoison commented on June 9, 2024

@dehann I generated the wrappers for amd, camd, colamd and ccolamd with #61 if you are still interested to add your code here.

from amd.jl.

dehann commented on June 9, 2024

Hi @amontoison , I looked at the PR and your ccolamd code seems to cover it. The code I have does roughly the same thing but doesn't have the error states etc. My code might also be a bit stale as it was written on Julia 0.6. I'd say go ahead with 61 thanks! That would also close this issue.

from amd.jl.

amontoison commented on June 9, 2024

Thanks for your answer @dehann. I merged #61, I will do a new release.

from amd.jl.

Recommend Projects