-
Notifications
You must be signed in to change notification settings - Fork 33
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
Synthea Team Updates for BFD-458 (Test with Synthea Data) #775
Closed
hadleynet
wants to merge
25
commits into
CMSgov:BFD-458-test-synthea-data-2
from
synthetichealth:BFD-458-test-synthea-data-2
Closed
Synthea Team Updates for BFD-458 (Test with Synthea Data) #775
hadleynet
wants to merge
25
commits into
CMSgov:BFD-458-test-synthea-data-2
from
synthetichealth:BFD-458-test-synthea-data-2
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
… for Part D contracts
hadleynet
requested review from
cbrunefearless,
jzulim and
karlmdavis
as code owners
September 21, 2021 20:20
…r each beneficiary together in that file.
…SERTS and UPDATES into two files. Ignore UPDATES for counting purposes in unit tests
Closing since this already merged from another branch |
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.
Change Details
This PR updates the
BFD-458-test-synthea-data-2
branch to account for all the RIF files that the Syntheabb-export
branch outputs.In order for these changes to function, the environment needs to have a set a
SYNTHEA_OUTPUT_DIR
environment variable that points to the Synthea RIF files. If Synthea is configured to export RIF, these files will be in{path}/synthea/output/bfd
, but your testing environment could also have a static synthetic dataset stashed somewhere (or none at all), hence the environment variable.Acceptance Validation
Given a set of synthetic RIF files, do the integration tests successfully pass for the
bfd-pipeline-rif-load
andbfd-server-war
projects?Feedback Requested
Changes to make, if necessary.
External References
Security Implications
The synthetic data generated by Synthea contains no privacy or security concerns or restrictions.
new software dependencies
altered security controls
new data stored or transmitted
security checklist is completed for this change
requires more information or team discussion to evaluate security implications