modify by discuss