-
Notifications
You must be signed in to change notification settings - Fork 3
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
DO NOT MERGE: Update Runtime to work with Berkeley schema (meta PR) #581
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…reventing-runtime-from-booting-with-berkeley-schema Switch to "Berkeley schema" and eliminate broken references preventing application startup
38 tasks
`berkeley`: Merge `main` into `berkeley` (to trigger GHA workflow)
Updating to a berkeley schema valid biosample json file
Add type to DataObject.
Added missing comma for DataObject example
…berkeley-20240717 Merge `main` into `berkeley` (July 17)
…lid-biosample `berkeley`: Update `test_endpoints.py` to fix some failing tests
…r-to-use-first-typecode-when-pattern-contains-multiple-typecodes Update minter to use first typecode when pattern contains multiple typecodes
…le-merging-main-into-berkeley Merge `main` into `berkeley` (routine merge on August 9, 2024)
…ime (#634) * Documentation expectations regarding merging `main` into `berkeley` * Fix typo * Clarify expectation * Clarify sentence * Fix typo
…#637) * Get ORCID base URL (production vs. sandbox) from environment variable * style: reformat --------- Co-authored-by: github-actions <[email protected]>
…ctionary-is-not-compatible-with-orcid-sandbox-environment Use ORCID environment-specific JWKS values and clean up existing comments
* Update test_gold_translator.py to add `type` * Update test_gold_translator.py remove extra bracket * Update test_gold_translator.py syntax fixes * chore: format * fix: pass `type` to constructor closes #600 * fix: make test idempotent * feat: more resilient test data loading * fix: failing `test_get_samp_taxon_id` * fix: `test_get_collection_date` requires timestampvalue type * fix: add type to _get_quantity_value for gold & submission portal translators * fix:_get_text_value * update:test_get_controlled_term_value with comment * fix:test_get_lat_lon types * add types to _get_env_term_value * add types to submission portal translator * fix:broken neon translator tests * add types for submission portal translator * update doi getter * feat: clear test dbs on `make down-test` * fix: `test_get_doi` * fix: partial fix for `test_get_dataset` * add types to test data yaml * add more types * add credit assoc * add person, study * add type for env_package * add types for mock submission object * add types to translator * skip three remaining failing tests --------- Co-authored-by: Donny Winston <[email protected]> Co-authored-by: Jing <[email protected]>
* update /workflows/activities to /workflows/workflow_executions * modify test submit workflow executions * remove api/v1/workflows dir * remove unused files in `api/v1` * formatting * update endpoints * feat: refactor find endpoints * feat: reproduce deprecated interface and use `deprectated` kwarg * chore: docstring update * fix: `find_workflow_execution_by_id` add tests for /workflow_executions/* endpoints * fix: unskip `test_submit_workflow_executions` --------- Co-authored-by: Jing <[email protected]> Co-authored-by: Donny Winston <[email protected]>
Merge `main` into `berkeley` (routine, August 23, 2024)
commenting out pytest skip
Updating test_submit_workflow_activities
update POST endpoint for workflows
…682-documentation-updates-for-berkeley Restore hyperlinks to "Metadata" section and modernize Runtime description
…activities `berkeley`: Unskip `test_submit_workflow_activities` test and make it work with Berkeley schema
…iles-are-being-skipped-by-mistake `berkeley`: Restore tests that were being skipped by mistake (oops)
…st_gold_api_ops Quick fixes for GOLD API client tests in `tests/test_ops/test_gold_api_ops.py`
Merge `main` into `berkeley` (routine, September 19, 2024)
The previous code made the invalid assumption that a given collection could only store documents of a single type. That constraint does not exist in databases conforming to the Berkeley schema (nmdc-schema v11+).
…integrity-notebook 576 berkeley updates ref integrity notebook
…-ensure_alldocs-fails-with-assertionerror-1 `berkeley`: Determine class ancestry per-type value (within collection) instead of per-collection (when generating `alldocs`)
…-data_objectsstudystudy_id-not-returning-correct-results Update field name to be consistent with Berkeley schema
I am closing this draft PR in order to see what effect closing it has on the symptoms described in issue #700. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I am creating this PR so that people can opt into receiving notifications whenever something gets merged into the
berkeley
branch.