default values for optimize

[CatherineH]

I think the default values for zOptimize should be numOfCycles = 0, algorithm=0, since these are the default values if you press enter in the optimization window in Zemax

[indranilsinharoy]

Hi Catherine,
Thanks for the input. You are right 👍 I think that will also save some typing. I should have done it that way the very first time :-). It will be updated in the next check in.

[CatherineH]

I did this in my fork. I also changed the zSetNSCSolve() to have default values for the pickup variables, because if you set the solve to variable it's annoying to have to set extraneous pickup info.

[indranilsinharoy]

That's awesome! I will look at the changes tonight and get back to you by tomorrow. Once we agree on the changes I would like to request you to send a pull request. Thanks a lot.

[indranilsinharoy]

Hi Catherine,
Good catch! Can you please send a pull request? Also, if possible can you please format argument list to zOptimize like this

def zOptimize(self, numOfCycles=0, algorithm=0, timeout=None)

instead of the current

def zOptimize(self, numOfCycles = 0, algorithm = 0, timeout=None)

Thanks a lot.

[indranilsinharoy]

Thanks, Catherine. Your changes have been merged.