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

Field interactions #54

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Field interactions #54

wants to merge 8 commits into from

Conversation

andraztori
Copy link
Collaborator

implement ability to specify specific field interaction parameters
the feature gets turned on with --ffm_interaction_matrix
then field weights can be expressed by --ffm_interaction field_id_1:field_id_2:weight

weight 0 means the interaction is fully masked out. default is weight 1.0 everywhere
field ids are sequential ids of fields as passed by field declaration parameters

WARNING: since fields are not named, this means that any change to order of field declarations requires careful adjustment of interactions too...

ping @adischw

@@ -10,8 +10,9 @@ edition = "2018"
[dependencies]
csv = "1.1.3"
# we need new version to enable static builds
fasthash = "0.4.0"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@andraztori why revert back to the specific revision? main branch has 0.4.0.

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

Successfully merging this pull request may close these issues.

2 participants