nci-cidc-api-modules 1.2.1__tar.gz → 1.2.3__tar.gz

{nci_cidc_api_modules-1.2.1/nci_cidc_api_modules.egg-info → nci_cidc_api_modules-1.2.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nci_cidc_api_modules
-Version: 1.2.1
+Version: 1.2.3
 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: requests>=2.32.5
 Requires-Dist: jinja2>=3.1.6
 Requires-Dist: certifi>=2025.8.3
 Requires-Dist: cloud-sql-python-connector[pg8000]>=1.18.4
-Requires-Dist: nci-cidc-schemas==0.28.2
+Requires-Dist: nci-cidc-schemas==0.28.3
 Dynamic: description
 Dynamic: description-content-type
 Dynamic: home-page
@@ -395,7 +395,7 @@ API authentication relies on _identity tokens_ generated by Auth0 to verify that
 
 - It is a well-formatted JWT.
 - It has not yet expired.
-- Its cryptographic signature is valid.
+- Its cryptographic signature is valid. 
 
 JWTs are a lot like passports - they convey personal information, they’re issued by a trusted entity, and they expire after a certain time. Moreover, like passports, JWTs **can be stolen** and used to impersonate someone. As such, JWTs should be kept private and treated sort of like short-lived passwords.
 

{nci_cidc_api_modules-1.2.1 → nci_cidc_api_modules-1.2.3}/README.md

@@ -354,7 +354,7 @@ API authentication relies on _identity tokens_ generated by Auth0 to verify that
 
 - It is a well-formatted JWT.
 - It has not yet expired.
-- Its cryptographic signature is valid.
+- Its cryptographic signature is valid. 
 
 JWTs are a lot like passports - they convey personal information, they’re issued by a trusted entity, and they expire after a certain time. Moreover, like passports, JWTs **can be stolen** and used to impersonate someone. As such, JWTs should be kept private and treated sort of like short-lived passwords.
 

{nci_cidc_api_modules-1.2.1 → nci_cidc_api_modules-1.2.3}/cidc_api/config/db.py

@@ -45,33 +45,7 @@ def get_sqlalchemy_database_uri(testing: bool = False) -> str:
         # Connect to the test database
         db_uri = environ.get("TEST_POSTGRES_URI", "fake-conn-string")
     elif not db_uri:
-        # TODO: figure out why IAM authentication causes an issue with the huge filter query
-        # db_uri = f"postgresql+pg8000://{environ.get('CLOUD_SQL_DB_USER')}:xxx@/{environ.get('CLOUD_SQL_DB_NAME')}"
-
-        secrets = get_secrets_manager(testing)
-
-        # If POSTGRES_URI env variable is not set,
-        # we're connecting to a Cloud SQL instance.
-
-        config: dict = {
-            "drivername": "postgresql",
-            "username": environ.get("CLOUD_SQL_DB_USER"),
-            "password": secrets.get(environ.get("CLOUD_SQL_DB_PASS_ID")),
-            "database": environ.get("CLOUD_SQL_DB_NAME"),
-        }
-
-        if environ.get("CLOUD_SQL_INSTANCE_NAME"):
-            socket_dir = environ.get("CLOUD_SQL_SOCKET_DIR", "/cloudsql/")
-
-            # If CLOUD_SQL_INSTANCE_NAME is defined, we're connecting
-            # via a unix socket from inside App Engine.
-            config["query"] = {"host": f'{socket_dir}{environ.get("CLOUD_SQL_INSTANCE_NAME")}'}
-        else:
-            raise RuntimeError(
-                "Either POSTGRES_URI or CLOUD_SQL_INSTANCE_NAME must be defined to connect " + "to a database."
-            )
-
-        db_uri = str(URL.create(**config).render_as_string(hide_password=False))
+        db_uri = f"postgresql+pg8000://{environ.get('CLOUD_SQL_DB_USER')}:xxx@/{environ.get('CLOUD_SQL_DB_NAME')}"
 
     assert db_uri
 
@@ -80,9 +54,7 @@ def get_sqlalchemy_database_uri(testing: bool = False) -> str:
 
 # Use SQLALCHEMY_ENGINE_OPTIONS to connect to the cloud but use uri for local db
 def cloud_connector(testing: bool = False):
-    return {}
-    # TODO: figure out IAM authentication
-    # if not testing and not environ.get("POSTGRES_URI"):
-    #    return {"creator": getconn}
-    # else:
-    #    return {}
+    if not testing and not environ.get("POSTGRES_URI"):
+        return {"creator": getconn}
+    else:
+        return {}

{nci_cidc_api_modules-1.2.1 → nci_cidc_api_modules-1.2.3}/cidc_api/models/files/facets.py

@@ -361,6 +361,11 @@ assay_facets: Facets = {
         "colorized image": FacetConfig(["/visium/colorized.tiff"]),
         "cytassist image": FacetConfig(["/visium/cytassist.tiff"]),
     },
+    "NULISA": {
+        "Metadata": FacetConfig(["/nulisa/metadata.csv", "Metadata for NULISA run"]),
+        "NPQ File": FacetConfig(["/nulisa/npq_file.csv", "NPQ file for NULISA run"]),
+        "Raw Counts File": FacetConfig(["/nulisa/raw_counts_file.csv", "Raw counts file for NULISA run"]),
+    },
     "mIHC": {
         "Samples Report": FacetConfig(["/mihc/sample_report.csv"], "Samples report for mIHC run"),
         "Multitiffs": FacetConfig(["/mihc/multitiffs.tar.gz"], "Multi Tiffs file from mIHC run"),

{nci_cidc_api_modules-1.2.1 → nci_cidc_api_modules-1.2.3}/cidc_api/models/models.py

@@ -26,6 +26,7 @@ __all__ = [
     "FileValidationErrors",
     "IngestionJobs",
     "JobFileCategories",
+    "ValidationConfigs",
     "TRIAL_APPENDIX_A",
     "REQUEST_LETTER",
     "ADMIN_FILE_CATEGORIES",
@@ -531,6 +532,7 @@ class IAMException(Exception):
 EXTRA_DATA_TYPES = [
     "participants info",
     "samples info",
+    "clinical_upload",
     "file_group",
 ]
 ALL_UPLOAD_TYPES = set(
@@ -3117,7 +3119,7 @@ class DownloadableFiles(CommonColumns):
                 cls.file_purpose.label(purp_col.key),
                 func.json_agg(cls.id).label(ids_col.key),
             )
-            .group_by(cls.trial_id, cls.data_category_prefix, cls.file_purpose)
+            .group_by(cls.trial_id, type_col, purp_col)
             .alias("id_bundles")
         )
         purpose_bundles = (
@@ -3429,7 +3431,7 @@ class IngestionJobs(CommonColumns):
     trial_id = Column(String, nullable=False)
     version = Column(Integer, nullable=False)
     pending = Column(Boolean, nullable=False, default=False)
-    start_date = Column(Date, nullable=True)
+    start_date = Column(DateTime, nullable=True)
 
     @staticmethod
     @with_default_session
@@ -3497,11 +3499,33 @@ class IngestionJobs(CommonColumns):
             .first()
         )
 
-    # TODO: figure out which users have access to which jobs
     @classmethod
     @with_default_session
     def get_open_jobs_for_user(cls, user: Users, session: Session = None) -> list["IngestionJobs"]:
-        return session.query(cls).filter(cls.status.notin_(["DRAFT"])).order_by(cls._created.desc()).all()
+        if user.role not in [CIDCRole.ADMIN.value, CIDCRole.CLINICAL_TRIAL_USER.value]:
+            return []
+        job_query = session.query(cls).filter(cls.status.notin_(["DRAFT"]))
+        if (
+            user.role != CIDCRole.ADMIN.value
+            and not session.query(Permissions)
+            .filter(
+                Permissions.granted_to_user == user.id,
+                Permissions.upload_type == "clinical_upload",
+                Permissions.trial_id == Permissions.EVERY,
+            )
+            .all()
+        ):
+            authorized_trials = (
+                session.query(Permissions)
+                .filter(
+                    Permissions.granted_to_user == user.id,
+                    Permissions.upload_type == "clinical_upload",
+                    Permissions.trial_id != Permissions.EVERY,
+                )
+                .all()
+            )
+            job_query = job_query.filter(cls.trial_id.in_(map(lambda x: x.trial_id, authorized_trials)))
+        return job_query.order_by(cls._created.desc()).all()
 
 
 class JobFileCategories(CommonColumns):
@@ -3579,3 +3603,23 @@ class FileValidationErrors(CommonColumns):
         if file_ids:
             session.query(cls).filter(cls.file_id.in_(file_ids)).delete(synchronize_session=False)
             session.commit()
+
+    @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()
+
+
+class ValidationConfigs(CommonColumns):
+    __tablename__ = "validation_configs"
+
+    data_label = Column(String, nullable=True)
+    validation = Column(String, nullable=True)
+    scope = Column(String, nullable=True)
+    active = Column(Boolean, nullable=False, default=True, server_default="true")
+
+    @classmethod
+    @with_default_session
+    def get_active_configs_by_scope(cls, scope: str, session=None) -> list["ValidationConfigs"]:
+        """Return all active ValidationConfigs rows for the given scope."""
+        return session.query(cls).filter(cls.scope == scope, cls.active.is_(True)).all()

{nci_cidc_api_modules-1.2.1 → nci_cidc_api_modules-1.2.3/nci_cidc_api_modules.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nci_cidc_api_modules
-Version: 1.2.1
+Version: 1.2.3
 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: requests>=2.32.5
 Requires-Dist: jinja2>=3.1.6
 Requires-Dist: certifi>=2025.8.3
 Requires-Dist: cloud-sql-python-connector[pg8000]>=1.18.4
-Requires-Dist: nci-cidc-schemas==0.28.2
+Requires-Dist: nci-cidc-schemas==0.28.3
 Dynamic: description
 Dynamic: description-content-type
 Dynamic: home-page
@@ -395,7 +395,7 @@ API authentication relies on _identity tokens_ generated by Auth0 to verify that
 
 - It is a well-formatted JWT.
 - It has not yet expired.
-- Its cryptographic signature is valid.
+- Its cryptographic signature is valid. 
 
 JWTs are a lot like passports - they convey personal information, they’re issued by a trusted entity, and they expire after a certain time. Moreover, like passports, JWTs **can be stolen** and used to impersonate someone. As such, JWTs should be kept private and treated sort of like short-lived passwords.
 

{nci_cidc_api_modules-1.2.1 → nci_cidc_api_modules-1.2.3}/nci_cidc_api_modules.egg-info/requires.txt

@@ -20,4 +20,4 @@ requests>=2.32.5
 jinja2>=3.1.6
 certifi>=2025.8.3
 cloud-sql-python-connector[pg8000]>=1.18.4
-nci-cidc-schemas==0.28.2
+nci-cidc-schemas==0.28.3

{nci_cidc_api_modules-1.2.1 → nci_cidc_api_modules-1.2.3}/requirements.modules.txt

@@ -20,4 +20,4 @@ requests>=2.32.5
 jinja2>=3.1.6
 certifi>=2025.8.3
 cloud-sql-python-connector[pg8000]>=1.18.4
-nci-cidc-schemas==0.28.2
+nci-cidc-schemas==0.28.3

{nci_cidc_api_modules-1.2.1 → nci_cidc_api_modules-1.2.3}/tests/test_api.py

@@ -441,6 +441,7 @@ def test_endpoint_urls(cidc_api):
         "/jobs/<int:job_id>",
         "/jobs/<int:job_id>/files",
         "/jobs/<int:job_id>/status",
+        "/jobs/<int:job_id>/error_report",
         "/jobs/trials/<string:trial_id>/current",
         "/permissions/",
         "/permissions/<int:permission>",