Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mutect 2: Mapping Update #150

Closed
rosibaj opened this issue Feb 11, 2021 · 2 comments
Closed

Mutect 2: Mapping Update #150

rosibaj opened this issue Feb 11, 2021 · 2 comments

Comments

@rosibaj
Copy link
Contributor

rosibaj commented Feb 11, 2021

  • Update ES Mapping to include the fields needed for Mutect2
  • There should be three fields for each of the running states. (running/complete/error)
  • Possibly need to migrate indices to new mapping since we are adding fields?
@Buwujiu
Copy link
Contributor

Buwujiu commented Feb 25, 2021

Tested in argo dev, I did the following:

  1. indexed PACA-CA using donor aggregator swagger index-program endpoint
  2. checked the mapping of the latest es index donor_centric_program_pacaca_re_37 in Kibana:
    GET /donor_centric_program_pacaca_re_37/_mapping
    new mutect fields are present in the mapping:
        "mutectCompleted" : {
          "type" : "byte"
        },
        "mutectFailed" : {
          "type" : "byte"
        },
        "mutectRunning" : {
          "type" : "byte"
        },

@Buwujiu Buwujiu added the v1.18.0 label Mar 4, 2021
@rosibaj
Copy link
Contributor Author

rosibaj commented Mar 8, 2021

working as expected all the way up to gateway/ui. LGTM. no bugs found! thanks @Buwujiu !

@rosibaj rosibaj closed this as completed Mar 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants