make some notes on impl