Skip to content

[LPD] [Correct] Add option to disable Oauth when using CalCat proxy

Thomas Kluyver requested to merge lpd-calcat-oauth-opt into master

Description

After some tweaks to the proxy code, this is now working, so you can configure ~/.config/pycalibration/cal_tools/restful_config.yaml like this:

calcat:
    use-oauth2: false
    base-api-url: http://max-exfl017.desy.de:9876/api/

The default is use-oauth2: true, so this doesn't change any behaviour without a change in config. There's no particular rush to add this, I just wanted to test the proxy in a real use case - and doing so quickly meant I found and fixed some major bugs.

How Has This Been Tested?

xfel-calibrate lpd CORRECT --slurm-mem 700 --request-time 2022-05-17T14:20:21 \
    --slurm-name correct_FXE_lpd_202201_p003073_r14 \
    --karabo-da LPD00 LPD01 LPD02 LPD03 LPD04 LPD05 LPD06 LPD07 LPD08 LPD09 LPD10 LPD11 LPD12 LPD13 LPD14 LPD15 \
    --mem-cells 512 --bias-voltage 250.0 --capacitor 5pf --photon-energy 9.2 --category 0 \
    --offset-corr --rel-gain --ff-map --gain-amp-map \
    --in-folder /gpfs/exfel/exp/FXE/202201/p003073/raw --run 14 \
    --karabo-id FXE_DET_LPD1M-1 \
    --out-folder /gpfs/exfel/data/scratch/kluyvert/lpd-corr-p3073-r14-proxy \
    --sequences 1

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist:

  • My code follows the code style of this project.

Reviewers

@schmidtp @maial

Edited by Thomas Kluyver

Merge request reports