Skip to content

Commit

Permalink
Finalize plots for paper
Browse files Browse the repository at this point in the history
Fixes #13
  • Loading branch information
hechth committed Nov 13, 2023
1 parent adf8208 commit c978dad
Show file tree
Hide file tree
Showing 8 changed files with 756 additions and 351 deletions.
123 changes: 98 additions & 25 deletions analysis/Python_scripts/chemical_composition_boxplot.ipynb

Large diffs are not rendered by default.

Binary file modified analysis/Python_scripts/chemical_composition_boxplot.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified analysis/Python_scripts/chemical_composition_boxplot2.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified analysis/Python_scripts/chemical_composition_boxplot2_N.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified analysis/Python_scripts/chemical_composition_boxplot_N.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified analysis/Python_scripts/scatter_plot.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
980 changes: 656 additions & 324 deletions analysis/Python_scripts/scatterplot.ipynb

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions analysis/Python_scripts/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ def has_atom(mol, atom):
def has_organic_atoms(mol):
# Check if the molecule contains any halogen atoms
for atom in mol.GetAtoms():
if atom.GetSymbol() in ['C', 'O', 'N', 'H']:
if atom.GetSymbol() in ['C', 'O', 'H']:
return True

return False
Expand All @@ -47,7 +47,7 @@ def append_classes(df, left_on):
"S": [has_atom(m, 'S') for m in molecules],
"P": [has_atom(m, 'P') for m in molecules],
"Si": [has_atom(m, 'Si') for m in molecules],
"C,O,N,H": [has_organic_atoms(m) for m in molecules],
"C,O,N,H": [has_organic_atoms(m) or has_atom(m, 'N') for m in molecules],
"N": [has_atom(m, 'N') for m in molecules],
})
merged_df = pd.merge(df, class_names, left_on=left_on, right_on='molname')
Expand Down

0 comments on commit c978dad

Please sign in to comment.