this should be in your constructor, not a separate call