pypeit / PypeIt-development-suite

8 stars 11 forks source link

Fix quick look dependency order, keck_deimos QL, and other changes #191

Closed badpandabear closed 2 years ago

badpandabear commented 2 years ago

This PR: Reverts the changes in https://github.com/pypeit/PypeIt-development-suite/pull/177 that introduced the keck_deimos/QL test but also unintentionally changed things so that reduce tests depended on quick look tests. This has been changed back so that quick look tests depend on reduce tests.

Re-implements the keck_deimos/QL test support by removing the assumption that all test setups run a "reduce" step. It also installs console_scripts in gen_kube_devsuite so that the pypeit_ql_keck_deimos script is installed in the cloud. Also fix the timing of tests for situations in which "run" is called multiple times per test.

Add a new --no_gui option to pypeit_test, and update gen_kube_devsuite to use this option. This allows for monitoring dev-suite progress in the cloud by using "log -f pod name".

Remove use of os.walk() and use pathlib instead because of reported issues.