updated ValueGeneratorRegistry so that factories are registered instead
of concrete generators