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

Problem with UCDP Armed Conflict Data Merge #14

Closed
rudloffprof opened this issue Nov 29, 2022 · 2 comments
Closed

Problem with UCDP Armed Conflict Data Merge #14

rudloffprof opened this issue Nov 29, 2022 · 2 comments

Comments

@rudloffprof
Copy link

An example:

library(preacesciencr)
test <- create_stateyears(system = "gw", subset_years = c(1946:2019))
test <- add_ucdp_acd(test, type = c("intrastate", "II"), issue = "both")
table(test$ucdpongoing)

This results in two state years out of ~10,000 coded as "1" for the ucdpongoing variable: Syria in 2016 and Syria in 2018 (in case this helps with diagnosis).

R version: 4.2.2
peacesciencer version: 1.0.0

I took a brief look at the source code for the add_ucdp_acd() function, and nothing was immediately apparent that would explain why only these two years for Syria would be coded correctly in the resulting data.

@svmiller
Copy link
Owner

Looking into this now, and this might be a case where I show how little I know or think about the UCDP data, but the "both" in the issue coding is filtering on the raw UCDP ACD data where incompatibility == 3 . Doing that, there are 18 cases and that's in the raw data. Selecting on those 18 cases where type_of_conflict >= 3, you get the two Syrian cases, and that's it.

As a matter of functionality, it looks like add_ucdp_acd() is behaving as it should. If there's a better, more informative way of processing the underlying UCDP ACD data, that could be happening too. Here's where I'll reiterate I'm primarily an inter-state conflict guy. :P

@rudloffprof
Copy link
Author

No, you are correct. I was reading issue = "both" as issue = c("territory", "government"), and missing that "both" refers to a logical "and" of both categories per conflict id in the underlying UCDP variable. Closing the issue - sorry for the false alarm!

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

No branches or pull requests

2 participants