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

Value weighting in ideology matching (1/2) #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ninjaofbacon
Copy link

@ninjaofbacon ninjaofbacon commented Sep 21, 2017

Created new variables to allow each ideology to have a different weight for different values. This is done because some ideologies, like anarcho-capitalism, have little to nothing to do with society, while others, such as fascism, are very tradition-oriented.

@TristanBomb
Copy link
Contributor

TristanBomb commented Sep 24, 2017

Actually, this is an excellent idea, and I'm a little ashamed I didn't think of it during my initial implementation! This solves a number of issues I had while trying to calibrate ideologies. I'll look it over and see about including it.

I'd like to apologize for some of the poorly-commented code you had to dig through. I would also like to apologize for the magic number 1.73856063 that I used to calibrate the axes. I have literally no idea how I calculated that.

One thing I'd like to bring up, though, is that you need to be careful with the weights to make sure that certain ideologies don't return systematically lower scores because they have lower weights. This could possibly be remedied by affecting how the final distance is calculated; I'll think on it a bit and see if i can come up with a decent formula.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants