Fixed Error In Forming RDM

[JoeyT1994]

Now account for the fact combiners is a directed data graph.

[mtfishman]

Glad to see this part can be simplified. Could you add a test for this function?

[JoeyT1994]

Glad to see this part can be simplified. Could you add a test for this function?

I have added a testing unit for this now!

[mtfishman]

Would Metis work instead of KaHyPar for the test? Unfortunately it is a bit awkward getting the automatic testing working with KaHyPar because of the installation issues on Windows.

[JoeyT1994]

Good idea, I have changed it and the tests appear to be passing with Metis so I will use that for the testing.

[mtfishman]

Could you add the line:

LinearAlgebra = "37e2e46d-f89d-539d-b4ee-838fcccc9c8e"

to test/Project.toml? That will allow the tests to automatically install the LinearAlgebra package.

[JoeyT1994]

LinearAlgebra = "37e2e46d-f89d-539d-b4ee-838fcccc9c8e"

[JoeyT1994]

Added

Could you add the line:

LinearAlgebra = "37e2e46d-f89d-539d-b4ee-838fcccc9c8e"

to test/Project.toml? That will allow the tests to automatically install the LinearAlgebra package.

Sure, I added this.

[mtfishman]

Thanks!