Cleanup builder finalization code
From !57 (comment 4098):
I don’t like _transfer_symmetry()
In builder
. This is
instance initialization that should go into __init__()
. Also,
it takes a whole builder as argument, while it only needs the
discrete symmetry.
But the problem is actually deeper and begins with code that I
wrote: the _finalized_*()
methods of Builder
should be turned
into *System.__init__(self, builder)
. Common functionality in
both __init__()
functions should be factored out into functions
like _process_cons_law
. If you don’t mind, I can do this
reorganization in a separate commit once you’ve merged this MR.