-
Notifications
You must be signed in to change notification settings - Fork 32
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
Update tutorials #35
Update tutorials #35
Conversation
@@ -155,6 +156,7 @@ def get_cross_nmi(df_feat: pd.DataFrame, **kwargs) -> pd.DataFrame: | |||
)[0] / (0.5 * (diag[x_feat] + diag[y_feat])) | |||
mutual_info.loc[y_feat, x_feat] = mutual_info.loc[x_feat, y_feat] = I_xy | |||
|
|||
mutual_info.fillna(0, inplace=True) # if na => no relation => set to zero |
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.
Just commenting on this as something we should discuss in a moment
Co-authored-by: Matthew Evans <[email protected]>
Co-authored-by: Matthew Evans <[email protected]>
Co-authored-by: Matthew Evans <[email protected]>
modnet/preprocessing.py
Outdated
@@ -157,7 +158,7 @@ def get_cross_nmi(df_feat: pd.DataFrame, **kwargs) -> pd.DataFrame: | |||
mutual_info.loc[y_feat, x_feat] = mutual_info.loc[x_feat, y_feat] = I_xy | |||
|
|||
mutual_info.fillna(0, inplace=True) # if na => no relation => set to zero | |||
return mutual_info | |||
return mutual_info, diag # diag can be useful for future elimination based on entropy without the need of recomputing the cross NMI |
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.
Could we just set the diagonal values in the dataframe instead of setting them to 1, so we don't need to return this? This change breaks the existing API otherwise
Example notebooks updates to v0.1.9
Plus a few accompanying inner updates (each has its corresponding commit).
materials
argument, but conservingstructures
argument for backward compatibilityChemEnvSiteFingerprint|GaussianSymmFuncmean G2_0.05
should beGaussianSymmFunc|mean G2_0.05
(as used in MP_2018.6 on figshare)