{"push_id":1441569191,"size":1,"distinct_size":1,"ref":"refs/heads/run_scheme","head":"b995e80a45657545e2a59fc5b9ceeec4e6c467a3","before":"b52c8d839544e5ba718095f35d508816ff78bfa7","commits":[{"sha":"b995e80a45657545e2a59fc5b9ceeec4e6c467a3","author":{"name":"Jared Langevin","email":"8879014254d99fb082f16fa4df4fb5bfaac6ff60@gmail.com"},"message":"Multiple adjustments to 'ecm_prep.py' for package execution\n\nMultiple changes to ‘ecm_prep.py’ were required to ensure proper\nexecution of certain ECM packages:\n\n- Added a ‘yrs_on_mkt’ attribute to Measure and MeasurePackage objects\nto ensure proper accounting for overlaps in a package’s contributing\nECMs. This attribute had previously been set in ‘run.py,’ and is now\nremoved from that routine.\n\n- Merged together the “competed choice parameters,”  \"secondary mseg\nadjustments,” and \"supply-demand adjustment\" data for ECMs that\ncontribute to a package using a new ‘update_dict’ function. Previously,\nthese data had been merged using the built-in dict ‘update()’ function;\nhowever, this function sometimes overwrote data from one contributing\nECM with new data from another.\n\n- Improved the identification of which package ECMs require further\npreparation via ‘ecm_prep.py’. A package ECM is flagged for further\npreparation when:\n    1) it is newly defined by the user,\n    2) its ‘contributing_ECMs’ and/or ‘benefits’ parameters have been\nedited from a previous definition, or\n    3) the definitions for any of the individual measures that\ncontribute to the package have been edited.\n\n- In cases where a package’s contributing measure objects do not have\nupdated definitions and have thus not been initialized via the\n‘prepare_measures’ function, these contributing measure objects are now\ninitialized as part of the ‘prepare_packages’ function.\n\n- Removed ‘add_uncovered_pkgupdates’ function and supporting tests (no\nlonger needed).\n\nMinor adjustments have been made to the tested inputs and outputs in\n‘ecm_prep_test.py’ and ‘run_test.py’ to accommodate the above changes.\n\nFinally, a missing market entry year has been added to one of the\npreviously posted ECM definitions.","distinct":true,"url":"https://api.github.com/repos/trynthink/scout/commits/b995e80a45657545e2a59fc5b9ceeec4e6c467a3"}]}
{"actor":{"display_login":"jtlangevin"}}