Skip to content

Conversation

@mgolosova
Copy link
Collaborator

New fields are added on stage 017 (adjustMetadata):

  • ami_tags;
  • ctag_format_step;
  • ami_tags_format_step.

(Partially) overrides #371.

We have `ctag` field with the current (last) AMI tag; but fully
understand the process, to which the given task belongs, we can only by
the whole chain of AMI tags (as they basically define the steps of data
processing).

This field is not required now for any use-case, but will allow more
accurate classification of tasks by steps and, consequently, let us
provide more adequate results in some use-cases.
New fields are added on stage 017 (adjustMetadata):
* ctag_format_step;
* ami_tags_format_step.

There are different ways to say one step from another:
* MC production step name (already exists as 'step_name' field);
* current AMI tag + output data format;
* chain of AMI tags + output data format.

The latter is supposed to be the most universal, but initially only the
first one was used, then for some cases the second was invented, and...
...and there's no good way to make things as they are supposed to be all
at once. So we need all the possible namings.

---

NOTE.
Step names are based on the output data formats. They can be taken from
task's metadata `output_formats` field, but it wasn't used before (when
we needed tasks classification by format-based-steps). However,
according to M.Borodin, field 'output_formats' and list of data formats,
derived from the list of dataset names, must be exactly the same (if
format is specified in 'output_formats', the task _will_ produce a
dataset in the given format).
No one will be interested in statictics related to these formats. In
theory.

---

NOTE
I'd say that "technical" formats are "LOG" and "TXT", but (according to
M.Borodin) TXT is a proper data format.
@mgolosova mgolosova merged commit 1183cd9 into master Jul 29, 2020
@mgolosova mgolosova deleted the data4es-017-steps-refined-history branch July 29, 2020 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants