Kevin

Results 11 comments of Kevin

There is a minor problem I don't know how to best fix. I need to select a path at the sites for the workers to write their partition, I am...

There is a minor problem I don't know how to best fix. I need to select a path at the sites for the workers to write their partition, I am...

As a side note, when I ran all the tests the following testcases fail, but work when executed in isolation: - `FederatedWorkerIntegrationCRUDTest.testWorkerAddedForMonitoring` - `FederatedColAggregateTest.testTernaryColSumDenseMatrix` - `FederatedCtableTest.federatedCtableSinglenode` (configuration [3]) - `FederatedFullAggregateTest.testTernarySumDenseMatrixCP`...

You have a point there, I also realized this recently when trying to also replace SP Instructions without duplicating code. I will weigh the options, if I come up with...

Sure, that is the number one priority of this rework, that is why I also would like to remove the dependency for the string representations of those instructions to be...

After a lot of consideration I propose the following: 1. Switch to the first implementation approach. The refactor makes the code more maintainable, because of less indentation and implementation in...

> Thanks for getting started on this issue @kev-inn. Unfortunately, I think the PR goes a bit in the wrong direction. We first want to do a basic cleanup of...

I am sorry that this PR got so large. I believe it is better to do it in one go, but I can also break it down so the review...

Not ready for review, found some problems with the spark instructions.

`ConcurrentModificationException` seems to be more frequent with this PR, I do not know what the reason is for that, but that is anyway a fix for the future. EDIT: this...