Importance scripts#56
Merged
Merged
Conversation
RomanSraj
force-pushed
the
importance_scripts
branch
from
97dc74b to
9a09645
Compare
heidmic
requested changes
Jun 23, 2025
Owner
There was a problem hiding this comment.
Can you please add some documentation regarding what this does, why it is interesting, and what one should account for when executing it?
Owner
There was a problem hiding this comment.
can we also use the short version here?
builds on #55 and probably only adds fanova?
Collaborator
Author
RomanSraj
force-pushed
the
importance_scripts
branch
from
cca2b01 to
c418e3a
Compare
heidmic
requested changes
Jun 23, 2025
Owner
heidmic
left a comment
heidmic
left a comment
There was a problem hiding this comment.
Please also include some explanation on file level for the usage of this script
RomanSraj
force-pushed
the
use-newest-sklearn
branch
from
625cca5 to
91c1629
Compare
heidmic
approved these changes
Jul 1, 2025
heidmic
pushed a commit
that referenced
this pull request
Aug 29, 2025
* Use latest sklearn and add respective requirements * Only use essential requirements; Add 3.10 requirements (for cmpbayes) * Revert config.json change * Add warning and readme for cmpbayes * Revert config changes * Make python 3.12 work with cmpbayes * Revert error message and readme for cmpbayes * Importance scripts (#56) * Add importance script * Directly use output.txt files for importance analysis * Add float mapping for f_anova * Revert "Only use essential requirements; Add 3.10 requirements (for cmpbayes)" This reverts commit 56a86d1. * Add description to importance scripts * Add same formatting as in base branch * Remove unused Ipython * Use short requirements.txt * Add file level description * Change suprb repo back to master in requirements * Add calculate ranges function (#57) * Change back cmpbayes to dpaetzel repo * Remove fanova from requirements due to the system dependencies and the rare usage in a single one time script --------- Co-authored-by: m.heider <michael.heider@informatik.uni-augsburg.de>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.