search

Keck-DataReductionPipelines / KPF-Pipeline

KPF-Pipeline
https://kpf-pipeline.readthedocs.io/en/latest/
Other
11 stars 2 forks source link

Increased nCPU and updated comment #759

Closed awhoward closed 8 months ago

awhoward commented 8 months ago

Minor updates related to the production QLP script and reprocessing recipe

awhoward commented 8 months ago

Okay, I meant for this to be in a different branch, but this one will work.

The latest commit fixes the bug below caused by missing L2 data. Two observations that this affected are 'KP.20230119.08074.44' and 'KP.20230415.83168.63'. I tested the plot_CCF_grid() method of the QLP again with the changes on a handful of other observations and I didn't detect any problems.

Screenshot 2023-12-16 at 9 31 09 AM
awhoward commented 8 months ago

A few minor updates to scripts/kpf_processing_progress.py:

  1. If enddate < startdate, then the dates are printed in reverse chronological order
  2. The current Git version of KPF-Pipeline can be specified with, e.g. '--current version 2.5'. This is helpful if you want to check if all files are up through some recent version that isn't necessarily the latest version.
  3. The script can now be called without 'python' as the first text on the command line: ./scripts/kpf_processing_progress.py 20231231 20230101 --current_version 2.5

This is in response to https://github.com/Keck-DataReductionPipelines/KPF-Pipeline/issues/765.