Skip to content
This repository was archived by the owner on Jul 13, 2025. It is now read-only.

add ParameterBuilderBase class for manual inclusion of parameters#119

Merged
Nikronic merged 6 commits intomasterfrom
dynamic_parameter_builder
Nov 22, 2023
Merged

add ParameterBuilderBase class for manual inclusion of parameters#119
Nikronic merged 6 commits intomasterfrom
dynamic_parameter_builder

Conversation

@Nikronic
Copy link
Copy Markdown
Owner

Fixes #110

Summary

This PR creates the abstract classes needed for insertion of parameters (model features) manually.

For adding a new parameter, just extend the ParameterBuilderBase class, and provide the possible values it can take and their importance. Then, everything is handled by _modifier classes.

Remarks

I should NOT have added the constants for invitation letter states, yet I forgot. in simple terms, we have a dictionary where keys are the string name of possible responses from the user which are actually the person sending the invitation letter. The values, are out domain expert knowledge of the importance of that response, analogous to XAI values.

These constants need to be moved to #112.

in simple terms, we have a dictionary where keys are the string name of possible responses from the user which are actually the person sending the invitation letter. The values, are out domain expert knowledge of the importance of that response, analguous to XAI values.
These is a value check for `_modifiers` where take a percent base value, and should return percent base value.
This method is directly used by `potential` and `probability`.
@Nikronic Nikronic merged commit f63d2f4 into master Nov 22, 2023
@Nikronic Nikronic deleted the dynamic_parameter_builder branch November 22, 2023 10:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Addition of some of the most important questions that were not being asked at all

1 participant