CartesianSampler option documentation/cleanup (#5394)
Have used this for a long time but realized many of my assumptions were wrong due to the lack of option docs.
Hopefully I can help future people, or future me by fixing them.
dump_pdb/dump_pdb_tag -- this just dumps a pdb at the beginning of the mover before anything happens. I don't think this is necessary.
removed an unused movemap that is instantiated for no reason
add some clarification to some options (aka remove some XRW_TODO)
update some new to make_shared
also updates the CartesianSampler to not apply frames across pose chains ( I think this is correct -- Cartesian sampler should be able to fix a chainbreak, (ie if C and N are too far away) but in that case rosetta will think the chains are the same)