added some comments todo