argument after keywords with defaults

[esheldon]

Didn't even know this was legal, need to fix this while maintaining backwards compat

https://github.com/LSSTDESC/descwl-shear-sims/blame/3990880af4976a25a0e205df290d8df6d0d66067/descwl_shear_sims/sim.py#L267

[beckermr]

It is due to the * at the start of the function signature. That entry is interpreted as a required keyword without a default AFAIK.

[esheldon]

thanks for the explanation.

I guess we can just leave it then

[beckermr]

I would not leave it IMHO. That keyword should come first in the list at minimum for clarity.

[mr-superonion]

I set it to None by default:

descwl-shear-sims/descwl_shear_sims/sim.py

Line 307 in 4f90565

coadd_bbox_cen_gs_skypos=None,

and change it to WORD_ORIGIN if it is None:

descwl-shear-sims/descwl_shear_sims/sim.py

Lines 398 to 399 in 4f90565

	if coadd_bbox_cen_gs_skypos is None:
	coadd_bbox_cen_gs_skypos = WORLD_ORIGIN

[mr-superonion]

In the future, we will enable user to input se_wcs. Let's open new issue for that.