Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quality metrics (QCs) query related work #211

Open
wants to merge 73 commits into
base: main
Choose a base branch
from

Conversation

dmichaels-harvard
Copy link
Contributor

@dmichaels-harvard dmichaels-harvard commented Jul 26, 2024

  • Nested QC (QualityMetric) development.
    • Added enable_nested (true) for qc_values item in quality_metric.json.
    • Removed array type from qc_values.value.type array in quality_metric.json.
  • Simplified Makefile for deploy1/1a/1b/2 for dcicsnovault 11.21.1 changes;
    related to allowing smaht-portal and cgap-portal to run at the same time locally.
  • Added scripts/opensearch-dashboard-start.bash and Makefile target opensearch-dashboard-starto
    now that we are using OpenSearch rather than ElasticSearch we seem to need this rather than Kibana.
  • Added demo directory with script to export some sample/demo from smaht-staging and import locally.
  • Added /files/upload_file_size endpoint (types/file.py); for use by smaht-submitr to determine if a file to
    upload has already been uploaded; and get its size as a side-effect; returns HTTP 200 if found otherwise 404);
    in particular we want to check if it has been uploaded but is still marked as 'uploading' because its md5 is
    still in the process of being computed (i.e. otherwise we could simply look at the file status an be done with it).
  • New protection of /ingestion-status endpoint (in ingestion/ingestion_status.py) for authenticated
    users only; and also limit /ingestion-status/{keys,keys_sorted,flush} to admin users only.

@dmichaels-harvard dmichaels-harvard requested review from willronchetti and removed request for willronchetti August 12, 2024 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant