From b236f79a16567a5a6616cae28a0d61fe13257434 Mon Sep 17 00:00:00 2001
From: "i.nueske" <i.nueske@indiscale.com>
Date: Mon, 16 Dec 2024 17:35:59 +0100
Subject: [PATCH] MNT: Ignore pylint error where the current code seems the
best solution:
- In export_related.py export() we do not care which Exception is raised & need to continue
- In import_from_xml.py import_xml() the File from which the _checksum is deleted is created within this method
- In table_export.py BaseTableExporter.collect_information(), the method called by _call_find_function is not set, so determining which exceptions may be raised is difficult
- table_importer.py TSVImporter.read_file() only exists for backward compatibility
---
src/caosadvancedtools/export_related.py | 2 +-
src/caosadvancedtools/import_from_xml.py | 2 +-
src/caosadvancedtools/table_export.py | 2 +-
src/caosadvancedtools/table_importer.py | 2 +-
4 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/caosadvancedtools/export_related.py b/src/caosadvancedtools/export_related.py
index 2114f388..c7f25c90 100755
--- a/src/caosadvancedtools/export_related.py
+++ b/src/caosadvancedtools/export_related.py
@@ -118,7 +118,7 @@ def export(cont, directory="."):
try:
el.download(target)
print("Downloaded:", target)
- except Exception:
+ except Exception: # pylint: disable=broad-exception-caught
print("Failed download of:", target)
invert_ids(cont)
diff --git a/src/caosadvancedtools/import_from_xml.py b/src/caosadvancedtools/import_from_xml.py
index 540091b0..7eeafa67 100755
--- a/src/caosadvancedtools/import_from_xml.py
+++ b/src/caosadvancedtools/import_from_xml.py
@@ -63,7 +63,7 @@ def import_xml(filename, rerun=False, interactive=True):
for el in cont:
if isinstance(el, db.File):
- el._checksum = None
+ el._checksum = None # pylint: disable=protected-access
target = os.path.join("downloads", el.path[1:])
if os.path.exists(target):
diff --git a/src/caosadvancedtools/table_export.py b/src/caosadvancedtools/table_export.py
index 1805419b..32191530 100644
--- a/src/caosadvancedtools/table_export.py
+++ b/src/caosadvancedtools/table_export.py
@@ -172,7 +172,7 @@ class BaseTableExporter(object):
self.info[e] = val
else:
self._append_missing(e, d)
- except Exception as exc:
+ except Exception as exc: # pylint: disable=broad-exception-caught
self._append_missing(e, d)
logger.error(exc)
# last resort: check if record has e as property:
diff --git a/src/caosadvancedtools/table_importer.py b/src/caosadvancedtools/table_importer.py
index b061092e..c2cb0250 100755
--- a/src/caosadvancedtools/table_importer.py
+++ b/src/caosadvancedtools/table_importer.py
@@ -585,5 +585,5 @@ class CSVImporter(TableImporter):
class TSVImporter(CSVImporter):
- def read_file(self, filename, **kwargs):
+ def read_file(self, filename, **kwargs): # pylint: disable=arguments-differ
return super().read_file(filename, sep="\t", **kwargs)
--
GitLab