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.

Sign up for GitHub

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

Jump to bottom

Error with country name #95

Closed
zejuanxiaoyu opened this issue Sep 9, 2021 · 1 comment
Closed

Error with country name #95

zejuanxiaoyu opened this issue Sep 9, 2021 · 1 comment

Comments

@zejuanxiaoyu
Copy link

zejuanxiaoyu commented Sep 9, 2021 •
edited
Loading

Hi, I find the 'N Korea' outputs 'KR', which should be 'KP'. 'KR' is for South Korea. Thank you very much!
@konstantinstadler
Copy link
Member

Thanks a lot for reporting, will be fixed in 0.7.6

konstantinstadler added a commit that referenced this issue Aug 3, 2022
@konstantinstadler
N Korea match North Korea #95
711d8fc
konstantinstadler added a commit that referenced this issue Aug 3, 2022
@konstantinstadler @Kajwan
@MohamedhBadr @jm-rivera
test new readme format (#112)
b2111b0 
* update changelog

* added link to example script for parsing data

* vscode ignore in gitignore

* Mamba GitHub actions (#98)

- switched github actions to development and production
- testing on different os

* IEA (#97)

* added IEA classification

* Added IEA to README.rst

Co-authored-by: Kajwan Rasul <[email protected]>

* fixed country data missing NA

* updated CHANGELOG for new release

* updated CHANGELOG for new release

* new version

* Added EXIOBASE 3 letter codes (#100)

Added 3 letter codes for EXIOBASE 1, 2 and 3 with various test cases

Co-authored-by: Konstantin Stadler <[email protected]>

* IEA update and bug fix (#109)

* wrapped spec_name in re.escape()

allows for countries to have names with regular expression characters, such as parenthesis or dots.

* bug in pd max_rows

Returned 'Pattern matched multiple keys' option error

* ran black properly now

Co-authored-by: Kajwan Rasul <[email protected]>

* update changelog

* pump version

* pump version

* Changed to Türkiye and North Macedonia - fix #110

* Fix Bassas Da India - fix #111

* black formating

* Add oecd dac (closes #101) (#102)

* Added OECD DAC code classification to country_data.tsv

* Add DACcode to README.rst

* Add DACcode test to test_functionality.py

* update test_functionality.py

Missing white space at 662 for black test to pass successfully

* Update country_data.tsv

* Update country_data.tsv

* N Korea match North Korea #95

* Converted md files to rst

* Allow regex in ISO2 - fix #92

* rerun notebooks

* regex for iso

* changelog and new version test

* removed rst from links

* fix markdown render for pypi

Co-authored-by: Kajwan <[email protected]>
Co-authored-by: Kajwan Rasul <[email protected]>
Co-authored-by: Mohamed Badr <[email protected]>
Co-authored-by: Jorge Rivera <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@konstantinstadler @zejuanxiaoyu