ispyb / ispyb-database-modeling

4 stars 3 forks source link

Add status for MR and SAD phasing #51

Open olofsvensson opened 4 years ago

olofsvensson commented 4 years ago

The AutoProcStatus table is used for tracking the status of the automatic data processing pipelines. We would now need a similar table / mechanism for tracking the status of MR and SAD phasing pipelines.

rhfogh commented 4 years ago

Without knowing anything about the details, I wonder if it might be possible/sensible to use something like a single, shared ProcessingPipelineTable instead of one for each type of processing?

olofsvensson commented 4 years ago

I think you are right Rasmus, however, it would mean quite some refactoring. I guess it depends on how much manpower is available for refactoring given the developments of the new backend.

KarlLevik commented 4 years ago

At Diamond we now use the AutoProcProgram columns processingStatus tinyint(1) (NULL (not started/unknown), 0 (failure), 1 (success)) and processingMessage varchar(255) to indicate the current status for all processing pipelines (including MR and phasing), although this doesn't give us the "status history" for a particular processing job.

Solange's comment on #32 makes it sound like you're already using AutoProcProgram instead of PhasingProgramRun, so I guess the ESRF are sort of also doing what we're doing (just with the addition of AutoProcStatus).

There's an interesting feature (SQL Standard) in MariaDB 10.3+ which allows you to add history to tables such as AutoProcProgram, see: https://mariadb.com/kb/en/temporal-data-tables/

stufisher commented 4 years ago

Yes, please use AutoProcProgram, we already have one duplicate of this table (PhasingProgram), lets not add another.

We need to add the fk from AutoProcProgram to datacollection to make it more useful, the link via AutoProcIntegration is not useful for other types of processing

See https://github.com/ispyb/ispyb-database-modeling/issues/32

olofsvensson commented 4 years ago

Hi @rhfogh , @KarlLevik and @stufisher , I just had a zoom meeting with Solange, Max and Gianluca (@antolinos couldn't participate), here are our conclusions: