Store CalCat requests/responses for reproducibility
5 unresolved threads
5 unresolved threads
Compare changes
+ 1
− 1
@@ -116,7 +116,7 @@ if "readthedocs.org" not in sys.executable:
This directs CalCat queries through a proxy server which records the responses and replays them if the same queries are made again. This aims to improve reproducibility, by avoiding the need to talk to an external service when reproducing corrections, without duplicating all of the query code in a 'pre' notebook.
The proxy server code is in the calparrot repository, which should be reviewed alongside this.
The proxy server is launched on each allocated correction node, using a shared SQLite database (relying on SQLite's file locking to coordinate access). The first node to make a given request adds a row with a response status of -1 before making the request; other nodes that see this will wait up to 15 seconds for a response in the database before making their own request.
The proxy server picks a random port to listen on, and writes this in a file which we read back (I've tried to come up with a more elegant way to do this, but I haven't found any I like so far) EDIT: now the proxy server runs princess as a subprocess. We use environment variables to modify the cal_tools.restful_config settings so that notebooks will use the proxy server.
Potential risk: if we one day rearrange the constant HDF5 files, updating CalCat, we won't be able to reproduce corrections from before, because the paths are saved. However, changing the root directory (currently /gpfs/exfel/d/cal/caldb_store) is relatively easy, as this is not part of the CalCat responses.
I have run an LPD correction using this; the results are under
/gpfs/exfel/data/scratch/kluyvert/lpd-corr-p2808-r136-parrotThe metadata directory (slurm_out_...) includes calparrot.sqlite, with 32 entries - light & dark conditions for each of 16 LPD modules.
For now, this will only affect LPD correction, since the other detector corrections all use cal_db_remote, which isn't affected by this. !738 (merged) switches ePix100 over, and we plan to switch all detectors eventually.