-
Notifications
You must be signed in to change notification settings - Fork 20
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
Parameter Tuning Code Integration #193
base: master
Are you sure you want to change the base?
Changes from all commits
1872c48
7e0a990
d5d0461
20d20bf
71aa43e
691673c
26178f9
a5b3205
0aeda95
46c87fc
5265c53
9b7e687
23d1070
0408a20
f1f58e7
3507475
dd0359f
ef15799
47dab1a
b3504b5
dfcd302
97a7d7b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -173,4 +173,14 @@ analysis: | |
# 'euclidean', 'manhattan', 'cosine' | ||
metric: 'euclidean' | ||
evaluation: | ||
# evaluation per dataset-goldstandard pair | ||
# evalution will not run unless ml include is set to true | ||
include: true | ||
# adds evaluation per algorithm per dataset-goldstandard pair | ||
# evalution per algortihm will not run unless ml include and ml aggregate_per_algorithm is set to true | ||
aggregate_per_algorithm: true | ||
# TODO: should we decouple parts of eval that involve ml | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Lots of coupling happening now. I put in a solution for now in config.py, but is it worth separating the functions into their own true/ false? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe deal with some of the coupling by giving warnings and stopping the flow rather than silently shutting things off |
||
# it will be good to seperate them otherwise if ml doesn't work then eval won't work at all | ||
# pca_chosen | ||
# ensemble | ||
# precisin and recall |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we are already calculating the node ensembles, should we give it to the user?