add comments for future to do