Skip to content

Updated coefficients and variables - Small file structure changes#76

Merged
yamilbknsu merged 4 commits intodevfrom
yep/small_fixes
Sep 22, 2025
Merged

Updated coefficients and variables - Small file structure changes#76
yamilbknsu merged 4 commits intodevfrom
yep/small_fixes

Conversation

@yamilbknsu
Copy link
Collaborator

This PR updates the coefficients of the logit models and finalizes documentation and file structure for v1.0.0.

It also closes issue #75

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the coefficients and model expressions for various logit models to improve their accuracy and adds new demographic variables for better model fitting. It also finalizes documentation and file structure for v1.0.0 release.

  • Updates model coefficients and expressions across multiple demographic models (mortality, marriage, education, etc.)
  • Adds new age bin and family size variables for enhanced model precision
  • Implements proper handling of geographic identifiers (geoid vs county assignment)
  • Creates configuration file for SF Bay Area example

Reviewed Changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
notebooks/DevTODO.md Updates task completion status for model coefficients
demos/variables.py Adds new age bin and marital status variables for updated models
demos/models/kids_moving.py Separates geoid and county assignment for proper geographic handling
demos/models/household_reorg.py Implements separate geoid and county assignment logic
demos/config.py Adds validation for required lcm_county_id column
data/sf_bay_example/calibrated_models_coefficients/*.yaml Updates model coefficients and expressions across all demographic models
configuration/demos_config_sfbay.toml Creates comprehensive configuration for SF Bay Area example

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

yamilbknsu and others added 3 commits September 22, 2025 16:01
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@yamilbknsu yamilbknsu merged commit 7890b3d into dev Sep 22, 2025
2 checks passed
@yamilbknsu yamilbknsu deleted the yep/small_fixes branch January 6, 2026 19:33
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.

Issue for SCAG application where CountyID and TAZ cannot be used simultaneously

2 participants