Open Mike-Heneghan opened 5 years ago
From checking another management command which supports the verbose flag --verbose 1
that all methods need to include self
to be able to use self.verbose
.
Updated all method definitions and invocations to include self
where appropriate.
Realised that the way the service_area_by_region_top_category_count
method was written required calling services_in_service_area_regions
every time a region report was required and only using a fraction of the data set.
This would not be performant as services_in_service_area_regions
is very time consuming and demanding.
Wrote a method which calls services_in_service_area_regions
once, prints the appropriate data and then uses the results for service_area_by_region_top_category_count
.
Added further print statements for improving clarity.
From the feedback it would be useful to do the following:
--verbose 1
flag.