initial revision
need commas between parameters