Just a thought, but if you want to do 36V, you need to use 12 cells in series, not 10. Each a123 cell has a nominal voltage of 3.3V, so with a 10s setup, your pack voltage will be 33V.
I have used a similar packaging concept for a123 cells, grouping four cells together in parallel. and held together with hot glue and G10 plates, with a threaded rod down the middle:


One thing you will find is that unless you leave some portion of the spot-welded tabs on the cells, you won't be able to easily solder connections directly to the cells. What I used instead is some small nickel-plated battery springs, available from Mouser or DigiKey. This actually makes for a pretty solid connection, once the plates are compressed down.
Once the four cells are connectied in parallel, they can be considered as one larger cell. They will self-equalize to the same level. I did 4s4p "stick" packs, each with a 4-channel LVC board on the end, but doing a full BMS function would require a second board per stick.

What I did was to group these 12V/9.2Ah "stick packs" together, in various configurations. Here's a 24s4p "six-pack" arrangement that I used to use on my Mariner, before switching to the 48V-based Cyclone setup:

Oh, I did check the label on my controller, and it does say 48V and 35A:

-- Gary