nci-cidc-api-modules 1.2.6__py3-none-any.whl → 1.2.8__py3-none-any.whl

cidc_api/models/models.py

@@ -28,6 +28,7 @@ __all__ = [
     "JobFileCategories",
     "ValidationConfigs",
     "TRIAL_APPENDIX_A",
+    "TRIAL_APPENDIX_A_CELL_THAT_ENDS_THE_HEADER",
     "REQUEST_LETTER",
     "ADMIN_FILE_CATEGORIES",
     "FINAL_JOB_STATUS",
@@ -82,7 +83,6 @@ from sqlalchemy import (
     Boolean,
     CheckConstraint,
     Column,
-    Date,
     DateTime,
     Enum,
     ForeignKey,
@@ -150,6 +150,10 @@ from ..shared.gcloud_client import (
 os.environ["TZ"] = "UTC"
 logger = get_logger(__name__)
 
+# pylint: disable=protected-access,not-callable,no-value-for-parameter,too-many-positional-arguments
+# pylint: disable=unsubscriptable-object,singleton-comparison,broad-exception-raised,redefined-builtin
+# pylint: disable=arguments-renamed,logging-fstring-interpolation,disable=comparison-with-callable
+
 
 class FileGroups:
     pass
@@ -3391,8 +3395,8 @@ class PreprocessedFiles(CommonColumns):
         """
         if job_id is not None:
             return query.filter_by(job_id=job_id)
-        else:
-            return query.filter(cls.job_id.is_(None))
+
+        return query.filter(cls.job_id.is_(None))
 
 
 INGESTION_JOB_STATUSES = [
@@ -3415,6 +3419,7 @@ INGESTION_JOB_COLORS = {
 }
 # TODO If have "CANCELLED" concept or other final status, add here
 FINAL_JOB_STATUS = ["PUBLISHED"]
+TRIAL_APPENDIX_A_CELL_THAT_ENDS_THE_HEADER = "Entity"
 
 
 class IngestionJobs(CommonColumns):
@@ -3455,16 +3460,16 @@ class IngestionJobs(CommonColumns):
         df = gcs_xlsx_or_csv_file_to_pandas_dataframe(GOOGLE_CLINICAL_DATA_BUCKET, appendix_a.object_url)
         categories = []
         headers_ended = False
-        for index, row in df.iterrows():
+        for _index, row in df.iterrows():
             cell = str(row.iloc[0])
             if headers_ended:
-                if not cell == "nan" and cell not in categories and cell != "Specialized_Data":
+                if cell != "nan" and cell not in categories:
                     categories.append(cell)
-            elif cell == "PATIENT-LEVEL DATA":
+            elif cell == TRIAL_APPENDIX_A_CELL_THAT_ENDS_THE_HEADER:
                 headers_ended = True
-        if "Data_Dictionary" not in categories:
+        if "data_dictionary" not in categories:
             # Ensure Data_Dictionary is always a required file category
-            categories.append("Data_Dictionary")
+            categories.append("data_dictionary")
         return categories
 
     @classmethod
@@ -3608,7 +3613,16 @@ class FileValidationErrors(CommonColumns):
     @classmethod
     @with_default_session
     def get_errors_for_job_sorted_by_file_id(cls, job_id: int, session: Session = None) -> list["FileValidationErrors"]:
-        return session.query(cls).filter(cls.job_id == job_id).order_by(cls.file_id.asc()).all()
+        errors = (
+            session.query(cls)
+            .join(PreprocessedFiles)
+            .filter(PreprocessedFiles.status == "current")
+            .filter(cls.job_id == job_id)
+            .order_by(cls.file_id.asc())
+            .all()
+        )
+
+        return errors
 
 
 class ValidationConfigs(CommonColumns):

{nci_cidc_api_modules-1.2.6.dist-info → nci_cidc_api_modules-1.2.8.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nci_cidc_api_modules
-Version: 1.2.6
+Version: 1.2.8
 Summary: SQLAlchemy data models and configuration tools used in the NCI CIDC API
 Home-page: https://github.com/NCI-CIDC/cidc-api-gae
 License: MIT license
@@ -29,7 +29,7 @@ Requires-Dist: python-dotenv>=1.1.1
 Requires-Dist: requests>=2.32.5
 Requires-Dist: sqlalchemy>=2.0.43
 Requires-Dist: werkzeug>=3.1.3
-Requires-Dist: nci-cidc-schemas==0.28.6
+Requires-Dist: nci-cidc-schemas==0.28.7
 Dynamic: description
 Dynamic: description-content-type
 Dynamic: home-page

{nci_cidc_api_modules-1.2.6.dist-info → nci_cidc_api_modules-1.2.8.dist-info}/RECORD

@@ -5,7 +5,7 @@ cidc_api/config/secrets.py,sha256=jRFj7W43pWuPf9DZQLCKF7WPXf5cUv-BAaS3ASqhV_Q,14
 cidc_api/config/settings.py,sha256=NsJbqW6Vqcz2f79xcAbk4th5tHne_I-RPCbKq_3hpz0,4427
 cidc_api/models/__init__.py,sha256=bl445G8Zic9YbhZ8ZBni07wtBMhLJRMBA-JqjLxx2bw,66
 cidc_api/models/migrations.py,sha256=gp9vtkYbA9FFy2s-7woelAmsvQbJ41LO2_DY-YkFIrQ,11464
-cidc_api/models/models.py,sha256=nc2SzB4p16Sna50IlJzBShcBjIqDp-9i28bYnNR66G0,146286
+cidc_api/models/models.py,sha256=8hfETKW-FN27aDmntprcj-4T59oB46oK9sdrll5tprs,146839
 cidc_api/models/schemas.py,sha256=6IE2dJoEMcMbi0Vr1V3cYKnPKU0hv9vRKBixOZHe88s,2766
 cidc_api/models/files/__init__.py,sha256=8BMTnUSHzUbz0lBeEQY6NvApxDD3GMWMduoVMos2g4Y,213
 cidc_api/models/files/details.py,sha256=sZkGM7iEV4-J6IDQCdiMV6KBDLbPxCOqUMaU3aY9rX8,65153
@@ -17,8 +17,8 @@ cidc_api/shared/file_handling.py,sha256=l4wiRkVJLL7QbCoODsLx-uki6Km8QoMmUlRVnUV9
 cidc_api/shared/gcloud_client.py,sha256=ETQ34qKtO7l0svVhiHXxXewiyNjwIMgoeVcqrg-Twtk,36902
 cidc_api/shared/jose.py,sha256=-qzGzEDAlokEp9E7WtBtQkXyyfPWTYXlwYpCqVJWmqM,1830
 cidc_api/shared/rest_utils.py,sha256=RwR30WOUAYCxL7V-i2totEyeriG30GbBDvBcpLXhM9w,6594
-nci_cidc_api_modules-1.2.6.dist-info/licenses/LICENSE,sha256=pNYWVTHaYonnmJyplmeAp7tQAjosmDpAWjb34jjv7Xs,1102
-nci_cidc_api_modules-1.2.6.dist-info/METADATA,sha256=Sak0ZXL0cKD_WTaBIDoRYQl3qwOZwYRBKWEH7XAKrKk,39538
-nci_cidc_api_modules-1.2.6.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-nci_cidc_api_modules-1.2.6.dist-info/top_level.txt,sha256=rNiRzL0lJGi5Q9tY9uSoMdTbJ-7u5c_D2E86KA94yRA,9
-nci_cidc_api_modules-1.2.6.dist-info/RECORD,,
+nci_cidc_api_modules-1.2.8.dist-info/licenses/LICENSE,sha256=pNYWVTHaYonnmJyplmeAp7tQAjosmDpAWjb34jjv7Xs,1102
+nci_cidc_api_modules-1.2.8.dist-info/METADATA,sha256=AZJg2SGHrtzfm5wNtIBAzSfqRyrJdPGwRtmFKEoWxTY,39538
+nci_cidc_api_modules-1.2.8.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+nci_cidc_api_modules-1.2.8.dist-info/top_level.txt,sha256=rNiRzL0lJGi5Q9tY9uSoMdTbJ-7u5c_D2E86KA94yRA,9
+nci_cidc_api_modules-1.2.8.dist-info/RECORD,,