starting to add real logging