From b16e00ba78242e0f79c83f7d4ea72c240bc8b090 Mon Sep 17 00:00:00 2001 From: Robert Sachunsky <38561704+bertsky@users.noreply.github.com> Date: Wed, 31 Jan 2024 12:35:26 +0100 Subject: [PATCH] workspace: downgrade log messages from info to debug --- src/ocrd/workspace.py | 38 +++++++++++++++++++------------------- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/src/ocrd/workspace.py b/src/ocrd/workspace.py index 633e45acf5..aa1c3cb4dd 100644 --- a/src/ocrd/workspace.py +++ b/src/ocrd/workspace.py @@ -183,21 +183,21 @@ def download_file(self, f, _recursion_count=0): try: file_path.relative_to(Path(self.directory).resolve()) # raises ValueError if not relative # If the f.local_filename exists and is within self.directory, nothing to do - log.info(f"'local_filename' {f.local_filename} already within {self.directory}, nothing to do") + log.debug(f"'local_filename' {f.local_filename} already within {self.directory} - nothing to do") except ValueError: # f.local_filename exists, but not within self.directory, copy it - log.info("Copying 'local_filename' %s to workspace directory %s" % (f.local_filename, self.directory)) + log.debug("Copying 'local_filename' %s to workspace directory %s" % (f.local_filename, self.directory)) f.local_filename = self.resolver.download_to_directory(self.directory, f.local_filename, subdir=f.fileGrp) return f if f.url: - log.info("OcrdFile has 'local_filename' but it doesn't resolve, try to download from set 'url' %s", f.url) + log.debug("OcrdFile has 'local_filename' but it doesn't resolve - trying to download from 'url' %s", f.url) elif self.baseurl: - log.info("OcrdFile has 'local_filename' but it doesn't resolve and no 'url', concatenate 'baseurl' %s and 'local_filename' %s", - self.baseurl, f.local_filename) + log.debug("OcrdFile has 'local_filename' but it doesn't resolve, and no 'url' - trying 'baseurl' %s with 'local_filename' %s", + self.baseurl, f.local_filename) f.url = '%s/%s' % (self.baseurl, f.local_filename) else: raise FileNotFoundError(f"'local_filename' {f.local_filename} points to non-existing file," - "and no 'url' to download and no 'baseurl' set on workspace, nothing we can do.") + "and no 'url' to download and no 'baseurl' set on workspace - nothing we can do.") if f.url: # If f.url is set, download the file to the workspace basename = '%s%s' % (f.ID, MIME_TO_EXT.get(f.mimetype, '')) if f.ID else f.basename @@ -253,7 +253,7 @@ def remove_file(self, file_id, force=False, keep_file=False, page_recursive=Fals else: raise Exception("File not locally available %s" % ocrd_file) else: - log.info("rm %s [cwd=%s]", ocrd_file.local_filename, self.directory) + log.debug("rm %s [cwd=%s]", ocrd_file.local_filename, self.directory) unlink(ocrd_file.local_filename) # Remove from METS only after the recursion of AlternativeImages self.mets.remove_file(file_id) @@ -322,11 +322,11 @@ def rename_file_group(self, old, new): with pushd_popd(self.directory): # create workspace dir ``new`` - log.info("mkdir %s" % new) + log.debug("mkdir %s" % new) if not Path(new).is_dir(): Path(new).mkdir() local_filename_replacements = {} - log.info("Moving files") + log.debug("Moving files") for mets_file in self.mets.find_files(fileGrp=old, local_only=True): new_local_filename = old_local_filename = str(mets_file.local_filename) # Directory part @@ -348,28 +348,28 @@ def rename_file_group(self, old, new): mets_file.ID = new_id # change file paths in PAGE-XML imageFilename and filename attributes for page_file in self.mets.find_files(mimetype=MIMETYPE_PAGE, local_only=True): - log.info("Renaming file references in PAGE-XML %s" % page_file) + log.debug("Renaming file references in PAGE-XML %s" % page_file) pcgts = page_from_file(page_file) changed = False for old_local_filename, new_local_filename in local_filename_replacements.items(): if pcgts.get_Page().imageFilename == old_local_filename: changed = True - log.info("Rename pc:Page/@imageFilename: %s -> %s" % (old_local_filename, new_local_filename)) + log.debug("Rename pc:Page/@imageFilename: %s -> %s" % (old_local_filename, new_local_filename)) pcgts.get_Page().imageFilename = new_local_filename for ai in pcgts.get_Page().get_AllAlternativeImages(): for old_local_filename, new_local_filename in local_filename_replacements.items(): if ai.filename == old_local_filename: changed = True - log.info("Rename pc:Page/../AlternativeImage: %s -> %s" % (old_local_filename, new_local_filename)) + log.debug("Rename pc:Page/../AlternativeImage: %s -> %s" % (old_local_filename, new_local_filename)) ai.filename = new_local_filename if changed: - log.info("PAGE-XML changed, writing %s" % (page_file.local_filename)) + log.debug("PAGE-XML changed, writing %s" % (page_file.local_filename)) with open(page_file.local_filename, 'w', encoding='utf-8') as f: f.write(to_xml(pcgts)) # change the ``USE`` attribute of the fileGrp self.mets.rename_file_group(old, new) # Remove the old dir - log.info("rmdir %s" % old) + log.debug("rmdir %s" % old) if Path(old).is_dir() and not listdir(old): Path(old).rmdir() @@ -1108,9 +1108,9 @@ def _crop(log, name, segment, parent_image, parent_coords, op='cropped', **kwarg if (not isinstance(segment, BorderType) or # always crop below page level not op in parent_coords['features']): if op == 'recropped': - log.info("Recropping %s", name) + log.debug("Recropping %s", name) elif isinstance(segment, BorderType): - log.info("Cropping %s", name) + log.debug("Cropping %s", name) segment_coords['features'] += ',' + op # create a mask from the segment polygon: segment_image = image_from_polygon(parent_image, segment_polygon, **kwargs) @@ -1143,7 +1143,7 @@ def _reflect(log, name, orientation, segment_image, segment_coords, segment_xywh segment_coords['angle'] += orientation # transpose, if (still) necessary: if not 'rotated-%d' % orientation in segment_coords['features']: - log.info("Transposing %s by %d°", name, orientation) + log.debug("Transposing %s by %d°", name, orientation) segment_image = transpose_image(segment_image, transposition) segment_coords['features'] += ',rotated-%d' % orientation return segment_image, segment_coords, segment_xywh @@ -1160,7 +1160,7 @@ def _rotate(log, name, skew, segment, segment_image, segment_coords, segment_xyw segment_coords['angle'] += skew # deskew, if (still) necessary: if not 'deskewed' in segment_coords['features']: - log.info("Rotating %s by %.2f°", name, skew) + log.debug("Rotating %s by %.2f°", name, skew) segment_image = rotate_image(segment_image, skew, **kwargs) segment_coords['features'] += ',deskewed' if (segment and @@ -1193,7 +1193,7 @@ def _scale(log, name, factor, segment_image, segment_coords, segment_xywh, **kwa segment_xywh['h'] *= factor # resize, if (still) necessary if not 'scaled' in segment_coords['features']: - log.info("Scaling %s by %.2f", name, factor) + log.debug("Scaling %s by %.2f", name, factor) segment_coords['features'] += ',scaled' # FIXME: validate factor against PAGE-XML attributes # FIXME: factor should become less precise due to rounding