| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
* Fix TCTracks.from_FAST duplicate loading from year loop * Refactor test_from_FAST: extend fixture to 2 years, remove separate regression test The test fixture FAST_test_tracks.nc now has year=[2025,2026] (only seeds_per_month is extended; track variables retain their n_trk dim). The existing len(tc_track.data)==5 assertion now acts as the regression check: the buggy year-loop code would return 5x2=10 tracks. The separate test_from_FAST_not_multiplied_by_year_dim (with its temporary-file scaffolding) is removed. * Update changelo --------- Co-authored-by: Simona Meiler <simona.meiler@usys.ethz.ch> Co-authored-by: Chahan Kropf <chahan.kropf@posteo.com>
|
@emanuel-schmid or @peanutfun, this should be a quick one to review. And will fix some of the currently failing tests. |
Sorry, something went wrong.
|
There are some administrative regions, which lead to an error when trying to create LitPop for them. Please find the code how to determine these below. `country_names = global_litpop.gdf.region_id.unique() info_ind = "NY.GDP.MKTP.CD" for cntry_iso in iso3a_list: failing_c = [] |
Sorry, something went wrong.
There was a problem hiding this comment.
Looks good to me. In any case an improvement! 🥇
Happy to merge.
Sorry, something went wrong.
| Credit Suisse's Global Wealth Report 2017 (household wealth)""" | ||
|
|
||
| WB_URBAN_LAND_MARKUP = 1.24 | ||
|
|
There was a problem hiding this comment.
why the name and what does it do?
Sorry, something went wrong.
| 14676399165833.0, # 15/05/2026 from WB website | ||
| ] | ||
| ) # Values updated by worldbank at different dates | ||
| ref_val_noland = ref_val / 1.24 # dito |
There was a problem hiding this comment.
Sorry, something went wrong.
| (applies to 'NW.PCA.*' only). Default: True. | ||
| (applies to 'NW.PCA.*' only). The world bank applies a 24% markup | ||
| to include the built-up (or urban) land value, thus this divides | ||
| by a factor 1.24. |
There was a problem hiding this comment.
this explains the WB_URBAN_LAND_MARKUP 😄 , add a reference
Sorry, something went wrong.
| Return requested variable for a country (cntry_iso) and a year (ref_year). | ||
| Download wealth accounting data from the World Bank API and return the requested | ||
| variable for a country (cntry_iso) and a year (ref_year). | ||
| https://datacatalog.worldbank.org/search/dataset/0042066 |
There was a problem hiding this comment.
is this url correct? wasn't it meant to be replaced?
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Changes proposed in this PR:
This PR fixes #1291 and #1293
PR Author Checklist
PR Reviewer Checklist