MeasureGroupComponent Codes Don't Reflect Multiple Populations about cqf-ruler HOT 7 CLOSED

Adding the population's identifier to the populationReport in FhirMeasureEvaluator.resolveGroupings gives the client more information on the details of the classification.

				MeasureReport.MeasureReportGroupPopulationComponent populationReport = new MeasureReport.MeasureReportGroupPopulationComponent();
				populationReport.setCount(count);
				populationReport.setCode(population.getCode());
				populationReport.setIdentifier(population.getIdentifier());

and yields MeasureReports that can be classified like this:

http://localhost:8080/cqf-ruler/baseDstu3/Patient/Patient-1121/_history/1
	patient is in initial-population
	patient is in numerator 1
	patient is in denominator 1
	patient is in numerator 2
	patient is in denominator 2
http://localhost:8080/cqf-ruler/baseDstu3/Patient/Patient-1128/_history/1
	patient is in initial-population
	patient is in denominator 1
http://localhost:8080/cqf-ruler/baseDstu3/Patient/Patient-1122/_history/1
	patient is in initial-population
	patient is in denominator 1

from cqf-ruler.

And lets me produce Cat-3 statistics for the population (albeit slowly);

	denominator 1 = 81
	denominator 2 = 36
	initial-population = 81
	numerator 1 = 36
	numerator 2 = 6

I forked a branch with this fix but evidently don't have karma to push it for a pull request.

from cqf-ruler.

I haven't figured out where, but the identifier seems to already be set. So this is just pilot error on my part. Closing...

from cqf-ruler.

Reopening because, for some reason I don't yet understand (maybe in the hot-code replacement during debugging?), the identifiers are no longer being set. Checking the codes still yields codes that don't distinguish between numbered numerators, denominators etc.

from cqf-ruler.

With more digging, I've located why I thought the identifier was already being set and it did have to do with hot code replacement while I was debugging, so the patch really needs to be made.

from cqf-ruler.

I don't see the above patch in the latest checkout. Is there a better way included or is there a reason why it is not?

from cqf-ruler.

Closing, thanks

from cqf-ruler.