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

import local council data #507

Merged
merged 38 commits into from
Apr 24, 2024
Merged

Conversation

struan
Copy link
Member

@struan struan commented Mar 13, 2024

Fixes #204

Copy link

codecov bot commented Mar 13, 2024

Codecov Report

Attention: Patch coverage is 40.10417% with 115 lines in your changes are missing coverage. Please review.

Project coverage is 80.39%. Comparing base (ed97a4a) to head (d478671).
Report is 33 commits behind head on 487-display-local-councils.

❗ Current head d478671 differs from pull request most recent head 3ef66ef. Consider uploading reports for the commit 3ef66ef to get more accurate results

Files Patch % Lines
hub/import_utils.py 0.00% 42 Missing ⚠️
hub/views/area.py 50.94% 20 Missing and 6 partials ⚠️
hub/management/commands/base_importers.py 26.66% 22 Missing ⚠️
utils/mapit.py 12.50% 14 Missing ⚠️
hub/mixins.py 0.00% 5 Missing and 2 partials ⚠️
hub/templatetags/hub_filters.py 66.66% 2 Missing ⚠️
hub/views/explore.py 60.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@                      Coverage Diff                       @@
##           487-display-local-councils     #507      +/-   ##
==============================================================
- Coverage                       82.41%   80.39%   -2.02%     
==============================================================
  Files                             104      106       +2     
  Lines                            3457     3565     +108     
  Branches                          353      370      +17     
==============================================================
+ Hits                             2849     2866      +17     
- Misses                            519      605      +86     
- Partials                           89       94       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@struan struan force-pushed the local-council-imports branch 4 times, most recently from 2201f0f to cbbf3ab Compare March 21, 2024 15:01
@struan struan force-pushed the local-council-imports branch 2 times, most recently from e45bb88 to 834888a Compare March 26, 2024 16:35
@struan struan force-pushed the 487-display-local-councils branch from d8f050a to 5fe3a22 Compare April 1, 2024 10:46
@struan struan force-pushed the local-council-imports branch from 332b429 to 2df9883 Compare April 1, 2024 12:09
@struan struan force-pushed the 487-display-local-councils branch from 0d146c4 to 165bd4d Compare April 2, 2024 15:44
@struan struan marked this pull request as ready for review April 2, 2024 15:49
@struan struan requested a review from zarino April 2, 2024 15:49
Copy link
Member

@zarino zarino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks ok, feature-wise. Just some wording changes.

hub/management/commands/import_council_mrp_data.py Outdated Show resolved Hide resolved
hub/management/commands/import_council_mrp_data.py Outdated Show resolved Hide resolved
hub/templates/hub/area/_json_data.html Outdated Show resolved Hide resolved
@struan struan force-pushed the local-council-imports branch from 2d8284f to b26c1f3 Compare April 8, 2024 16:34
@struan struan changed the title WIP: import local council data import local council data Apr 9, 2024
@struan struan requested a review from zarino April 9, 2024 15:43
Copy link
Member

@zarino zarino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A lot’s changed in here! But with those wording changes all made, I think everything else now looks good to me 👍

@struan struan force-pushed the 487-display-local-councils branch from a5ea6d8 to 7a1e49b Compare April 24, 2024 08:57
@struan struan force-pushed the local-council-imports branch from 751e3a6 to 25002b6 Compare April 24, 2024 09:00
struan added 10 commits April 24, 2024 10:02
total emissions for council
emissions cluster
filters councils based on the area type to avoid many missing councils
messages
Also tidies up some of the formatting of averages for integer categories
and enables integers to be dates (mostly for years) that do not get
commas in them.
Include a link to the declaration if we have it and details on the scope
of the declaration.
struan added 25 commits April 24, 2024 10:02
use a lookup for the cons column if present
@struan struan force-pushed the local-council-imports branch from 25002b6 to 28f3abf Compare April 24, 2024 09:03
struan added 2 commits April 24, 2024 10:06
move range display into include
allow a range related dataset for datasets
@struan struan merged commit 3ef66ef into 487-display-local-councils Apr 24, 2024
10 checks passed
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.

Local authority climate actions (emergency declaration, net zero target dates, plans)
2 participants