Closes #3632: refactor LinalgMsg to remove registerND annotation

[ajpotts]

Refactors LinalgMsg to remove registerND annotation and replace with @arkouda.instantiateAndRegister.

@drculhane did all the work on this one, I'm just helping to submit the PR.

Closes #3632: refactor LinalgMsg to remove registerND annotation

[drculhane]

(1) You made excellent improvements; thanks.
(2) I'm having unusual problems making and testing this. I'm assuming that once again, it's most likely my VM. I'll probably post another comment with a summary later today.

[drculhane]

Okay, here's my follow-up.

(1) Had to increase my VM RAM to 32GB to make with max-array-rank of 3. My RevisedDraft3422 would make in 24GB (but not 16). I have no explanation for this.

(2) Some tests hung with max-array-rank < 3. With 3, all pass (or skip) except for two (in client_test.py): test_client_get_config and test_client_array_dim_cmd_error , which are hardcoded to expect max-array-rank of 1.

Those client_test.py tests are fixed in RevisedDraft3422, but when Amanda suggested splitting it into smaller PRs for evaluation, it slipped my mind that client_test.py needed to be part of the first one (and I'm not sure that this belongs in client_test anyway, but I don't have strong feelings about it).

[ajpotts]

I did end up making some minor changes to LinalgMsg to streamline code a little. I also changed one of the tests in client_test.py. Let me know what you think.

[jeremiah-corrado]

looks good to me! I have a couple of (very optional) suggestions on how to further simplify the server code:

[drculhane]

There's one of my "oops, I should have removed that" comments at line 2260 in numeric.py, but if that makes it through this PR, we'll take it out in the next one.

[stress-tess]

A few small suggestions, but I don't see any logic issues!