Skip to content

Commit f97b24e

Browse files
mamtsingquic-mamta
authored andcommitted
set log_level and dump_logs flag
Signed-off-by: Mamta Singh <[email protected]>
1 parent ce14058 commit f97b24e

File tree

4 files changed

+20
-22
lines changed

4 files changed

+20
-22
lines changed

QEfficient/cloud/finetune.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
#
66
# -----------------------------------------------------------------------------
77

8-
import logging
98
import random
109
import warnings
1110
from typing import Any, Dict, Optional, Union
@@ -41,8 +40,6 @@
4140
from QEfficient.utils._utils import login_and_download_hf_lm
4241
from QEfficient.utils.logging_utils import logger
4342

44-
logger.setLevel(logging.INFO)
45-
4643
# Try importing QAIC-specific module, proceed without it if unavailable
4744
try:
4845
import torch_qaic # noqa: F401
@@ -330,6 +327,9 @@ def main(peft_config_file: str = None, **kwargs) -> None:
330327
dataset_config = generate_dataset_config(train_config.dataset)
331328
update_config(dataset_config, **kwargs)
332329

330+
logger.prepare_dump_logs(train_config.dump_logs)
331+
logger.setLevel(train_config.log_level)
332+
333333
setup_distributed_training(train_config)
334334
setup_seeds(train_config.seed)
335335
model, tokenizer = load_model_and_tokenizer(train_config, dataset_config, peft_config_file, **kwargs)

QEfficient/cloud/infer.py

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -311,6 +311,13 @@ def main(
311311
action="store_true",
312312
help="pass to print info logs",
313313
)
314+
parser.add_argument(
315+
"--log_level",
316+
"--log-level",
317+
type=int,
318+
default=20,
319+
help="set the Log level {NOTSET:0, DEBUG:10, INFO:20, WARNING:30, ERROR:40, CRITICAL:50}",
320+
)
314321
parser.add_argument(
315322
"--full_batch_size",
316323
"--full-batch-size",
@@ -353,6 +360,11 @@ def main(
353360
)
354361
compiler_options_dict[key] = value
355362
if args.verbose:
356-
logger.setLevel(logging.INFO)
363+
logger.prepare_dump_logs(args.verbose)
364+
if args.log_level:
365+
logger.setLevel(args.log_level)
366+
else:
367+
logger.setLevel(logging.INFO)
357368
del args.verbose # type: ignore
369+
del args.log_level # type: ignore
358370
main(**args.__dict__, **compiler_options_dict)

QEfficient/finetune/configs/training.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@
44
# SPDX-License-Identifier: BSD-3-Clause
55
#
66
# -----------------------------------------------------------------------------
7+
8+
import logging
79
from dataclasses import dataclass
810

911

@@ -107,3 +109,4 @@ class TrainConfig:
107109
opByOpVerifier: bool = False
108110

109111
dump_logs: bool = True
112+
log_level: str = logging.INFO

QEfficient/utils/logging_utils.py

Lines changed: 1 addition & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -56,21 +56,6 @@ def create_logger() -> logging.Logger:
5656
ch.setFormatter(QEffFormatter())
5757
logger.addHandler(ch)
5858

59-
dump_logs = True
60-
if dump_logs:
61-
logs_path = os.path.join(ROOT_DIR, "logs")
62-
if not os.path.exists(logs_path):
63-
os.makedirs(logs_path, exist_ok=True)
64-
file_name = f"log-file-{datetime.now().strftime('%Y-%m-%d_%H-%M-%S')}" + ".txt"
65-
log_file = os.path.join(logs_path, file_name)
66-
67-
# create file handler and set level
68-
fh = logging.FileHandler(log_file)
69-
fh.setLevel(logging.INFO)
70-
formatter = logging.Formatter("%(levelname)s - %(name)s - %(message)s")
71-
fh.setFormatter(formatter)
72-
logger.addHandler(fh)
73-
7459
return logger
7560

7661

@@ -85,8 +70,7 @@ def log_rank_zero(self, msg: str, level: int = logging.INFO) -> None:
8570
return
8671
self.log(level, msg, stacklevel=2)
8772

88-
89-
""" def dump_logs(self, dump_logs=True):
73+
def prepare_dump_logs(self, dump_logs=False):
9074
if dump_logs:
9175
logs_path = os.path.join(ROOT_DIR, "logs")
9276
if not os.path.exists(logs_path):
@@ -100,7 +84,6 @@ def log_rank_zero(self, msg: str, level: int = logging.INFO) -> None:
10084
formatter = logging.Formatter("%(levelname)s - %(name)s - %(message)s")
10185
fh.setFormatter(formatter)
10286
logger.addHandler(fh)
103-
"""
10487

10588

10689
logging.setLoggerClass(CustomLogger)

0 commit comments

Comments
 (0)