Bugfix/numerator obs values

[RohitKandimalla]

Pull requests into cqm-execution require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.

Case: While generating observation results for each episode, we verify if the episode is a part of Numerator or Denominator, When Episode is not a part of Denominator, then the first index of observation_values array for that episode should be defaulted to 0. Bug: Instead of adding 0 at the first index, we are replacing the existing value, which could be the value of Numerator Observation.

https://jira.cms.gov/browse/MAT-6623 Screenshot of the contents from This story

Submitter:

• [x] This pull request describes why these changes were made.
• [x] Internal ticket for this PR:
• [ ] Internal ticket links to this PR
• [ ] Code diff has been done and been reviewed
• [x] Tests are included and test edge cases
• [x] Tests have been run locally and pass

Reviewer 1:

Name:

• [ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
• [ ] The tests appropriately test the new code, including edge cases
• [ ] You have tried to break the code

Reviewer 2:

Name:

• [ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
• [ ] The tests appropriately test the new code, including edge cases
• [ ] You have tried to break the code

[dczulada]

@RohitKandimalla is it possible to post the contents of the JIRA issue (https://jira.cms.gov/browse/MAT-6623 ) in this pull request? I'm having trouble understanding what this is trying to fix.

[RohitKandimalla]

@RohitKandimalla is it possible to post the contents of the JIRA issue (https://jira.cms.gov/browse/MAT-6623 ) in this pull request? I'm having trouble understanding what this is trying to fix.

Sure I add a screenshot, so that it will easier to understand c&p the contents. The over all issue is that Numerator Observations results are not returned by cqm-execution.

Technically, cql-execution is returning us the num_obs values, but during the construction of observation_values array, we are overriding existing values instead of unshifting the array.