Skip to content

Commit

Permalink
Log additional exclude regex later
Browse files Browse the repository at this point in the history
Signed-off-by: Tyler Gu <[email protected]>
  • Loading branch information
tylergu committed Feb 26, 2024
1 parent 98c8e3f commit 0cff773
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions acto/post_process/post_diff_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -705,9 +705,6 @@ def post_process(self, workdir: str, num_workers: int = 1):
def check(self, workdir: str, num_workers: int = 1):
"""Check the post process result"""
logger = get_thread_logger(with_prefix=True)
logger.info(
"Additional exclude paths: %s", self.config.diff_ignore_fields
)
trial_dirs = glob.glob(os.path.join(workdir, "trial-*"))

with open(self.config.seed_custom_resource, "r", encoding="utf-8") as f:
Expand Down Expand Up @@ -740,6 +737,10 @@ def check(self, workdir: str, num_workers: int = 1):
else:
self.config.diff_ignore_fields.extend(diff_skip_regex)

logger.info(

Check warning on line 740 in acto/post_process/post_diff_test.py

View workflow job for this annotation

GitHub Actions / coverage-report

Missing coverage

Missing coverage on lines 725-740
"Additional exclude paths: %s", self.config.diff_ignore_fields
)

processes = []
for i in range(num_workers):
p = multiprocessing.Process(
Expand Down

0 comments on commit 0cff773

Please sign in to comment.