Skip to content

Commit

Permalink
Add logger message when featurization parameters are reset
Browse files Browse the repository at this point in the history
  • Loading branch information
cjmcgill committed Sep 10, 2021
1 parent 3b779bf commit 0ce2bf2
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
10 changes: 9 additions & 1 deletion chemprop/features/featurization.py
Original file line number Diff line number Diff line change
@@ -1,8 +1,11 @@
from typing import List, Tuple, Union
from itertools import zip_longest
import logging

from rdkit import Chem
import torch
import numpy as np

from chemprop.rdkit import make_mol

class Featurization_parameters:
Expand Down Expand Up @@ -47,10 +50,15 @@ def __init__(self) -> None:
PARAMS = Featurization_parameters()


def reset_featurization_parameters() -> None:
def reset_featurization_parameters(logger: logging.Logger = None) -> None:
"""
Function resets feature parameter values to defaults by replacing the parameters instance.
"""
if logger is not None:
debug = logger.debug
else:
debug = print
debug('Setting molecule featurization parameters to default.')
global PARAMS
PARAMS = Featurization_parameters()

Expand Down
2 changes: 1 addition & 1 deletion chemprop/train/cross_validate.py
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ def cross_validate(args: TrainArgs,
args.save(os.path.join(args.save_dir, 'args.json'), with_reproducibility=False)

#set explicit H option and reaction option
reset_featurization_parameters()
reset_featurization_parameters(logger=logger)
set_explicit_h(args.explicit_h)
set_reaction(args.reaction, args.reaction_mode)

Expand Down

0 comments on commit 0ce2bf2

Please sign in to comment.