Colin B. Macdonald

Results 291 comments of Colin B. Macdonald

This seems like a reasonable plan to me. I think having organizations as parents to projects are a good idea: having let one of my projects languish recently, I now...

> If allow-blank = true, the checkbox input is marked as incorrect, not invalid. Is it possible to setup a question where the correct answer is to leave them all...

@alexvong243f after #1237 there are only 3 callers of the "list-of-lists" form: I think I will port them all to "flat+shape" (currently the underscore version). ``` @sym/private/elementwise_op.m: 'return _make_2d_sym(g, shape=q.shape)'...

Yep, vertcat is wrong: ``` >> A = sym(zeros(0,3)) A = (sym) [] (empty 0×3 matrix) >> [A; A] ans = (sym) [] (empty 0×0 matrix) ```

> In any case, I think we should keep the high-level function so that we use the low-level function only when we have to. I think it's quite common for...

How about `make_2d_sym_list_of_lists` flattens and then calls `_make_2d_sym`? That seems the easiest way to keep both. However, I think every caller of `make_2d_sym_list_of_lists` will need to think very carefully if...

Merging this for now, above decisions not really resolved but I don't think this hurts

Hmmm, it does give the error when run as `vertcat(z03, z04)` but not as `[z03; z04]`. Sounds like "Someone Else's Problem" (upstream). I'll add this zero case to the existing...

I didn't dig any deeper. Do you mean "compatibility with Matlab"? I don't know, have not tried it in Matlab. You can certainly file upsteam if you'd like.

I guess that's right, especially if that is convenient for other code. I'm not really concerned about them being errors either: anyone hitting this In Real Life has likely had...