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

Improve repo style, interface and add UK local authority breakdowns #62

Merged
merged 40 commits into from
Dec 18, 2024

Conversation

nikhilwoodruff
Copy link
Contributor

@nikhilwoodruff nikhilwoodruff commented Dec 10, 2024

Fixes #61

@nikhilwoodruff nikhilwoodruff added the enhancement New feature or request label Dec 10, 2024
@nikhilwoodruff nikhilwoodruff self-assigned this Dec 10, 2024
Copy link

@anth-volk anth-volk left a comment

Choose a reason for hiding this comment

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

I'm trying out Conventional Comments (link) in this review; feel free to let me know what you think.

There are only two blocking comments I had, one due to a potential error driven by referential equality and one around whether or not the docs successfully built. I've left various other nitpicks and suggestions, but I don't think these are vital to address in this PR, especially if (as it seems) this is majorly blocking for the local area constituency work.

README.md Outdated Show resolved Hide resolved
docs/_config.yml Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
policyengine/utils/maps.py Show resolved Hide resolved
policyengine/utils/huggingface.py Show resolved Hide resolved
policyengine/utils/charts.py Outdated Show resolved Hide resolved
policyengine/utils/charts.py Outdated Show resolved Hide resolved
@nikhilwoodruff nikhilwoodruff changed the title Add local authority breakdowns Improve repo style, interface and add UK local authority breakdowns Dec 17, 2024
@nikhilwoodruff nikhilwoodruff merged commit 8b5af39 into main Dec 18, 2024
2 checks passed
@nikhilwoodruff nikhilwoodruff deleted the local-authority-breakdowns branch December 18, 2024 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add local authority breakdowns
3 participants