Comment (by William Gabella):

I agreed to pull and review, test as well as I can. Some kind of test suite would be nice.

See Steve’s other pull request on the Python2 version with the --test switch on “sim” which will be changed to --dryrun . Might be a nice way to test sim over a broad range of options. See tickets #2482 and 2483,

https://bitbucket.org/einsteintoolkit/tickets/issues/2482/inconsistency-in-simfactory-submission

https://bitbucket.org/einsteintoolkit/tickets/issues/2483/simfactory-job-parameters-are-not

Zach shared a Python3 “lint” er at https://lgtm.com/projects/b/simfactory/simfactory2/?mode=list

which as of now shows 4 errors, 27 warnings, 161 recommendations. Overall code quality “D” Unclear to me whether this is the Steve Python3 code or the current Python2 code.

Also our current Simfactory is officially Simfactory2; that does not mean Python2. There is a moribund project Simfactory3 out there.

--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2335/switch-simfactory-to-use-python3