-
Notifications
You must be signed in to change notification settings - Fork 40
BFD-4693: Basis Profile Framework #3162
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
Merged
clewellyn-nava
merged 58 commits into
master
from
clewellyn-nava/BFD-4693/basis-profile-framework
Aug 10, 2026
Merged
Changes from 56 commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
2141c34
initial pass, need to fix pom
clewellyn-nava 3bafdf4
Actually commit initial attempt
clewellyn-nava a1703d7
FhirTrimmer implementation that is too slow
clewellyn-nava b3962d9
Fix unused implementation
clewellyn-nava b10090d
Merge branch 'master' into clewellyn-nava/BFD-4693/basis-profile-fram…
clewellyn-nava dc19bf1
Pass mvn clean verify
clewellyn-nava 014ee0b
Second attempt, also a failure
clewellyn-nava 069e393
Fix validation issues, this code will all be removed later anyway
clewellyn-nava b33fdc7
Start the yaml attempt
clewellyn-nava 677fa76
Copy dictionary files into server
clewellyn-nava a4de9bf
Small changes
clewellyn-nava e3d19c9
Merge branch 'master' into clewellyn-nava/BFD-4693/basis-profile-fram…
clewellyn-nava ed9d267
Add RFCS
clewellyn-nava 37a2f86
Fix checkmark errors, also start work on BasisProfile data structure
clewellyn-nava f500884
Fix sonarqube errors
clewellyn-nava 90c22a7
Fix sonarqube errors again
clewellyn-nava cd0bcf7
Continued iteration and simplification of tests
clewellyn-nava 34df2e9
Merge branch 'master' into clewellyn-nava/BFD-4693/basis-profile-fram…
clewellyn-nava 97d9c5d
Use symlinks for map files
clewellyn-nava 3a2dc77
Automatic changes
clewellyn-nava 9e12f81
Making some symlink changes
clewellyn-nava 65e4f35
Initial BasisProfileMap generation
clewellyn-nava 3ebc60c
Continued iteraction on FhirTrimmer
clewellyn-nava 033eba1
removeChild being called on Base is bad
clewellyn-nava 4cd3349
Commit for sanity
clewellyn-nava 743334f
Allow AI to attempt the problem, way too much reflection
clewellyn-nava e78b124
Commit work before cleaning repo
clewellyn-nava e0722d4
Merge branch 'master' into clewellyn-nava/BFD-4693/basis-profile-fram…
clewellyn-nava 6236a66
Massive refactor, work in progress
clewellyn-nava 1a65443
Continued file changes, with public and checkmark suppressions
clewellyn-nava c6d9b0f
Still moving files part 55
clewellyn-nava bf311ca
Still moving files part 56
clewellyn-nava a4abc3e
Merge branch 'master' into clewellyn-nava/BFD-4693/basis-profile-fram…
clewellyn-nava ca65f84
Finish initial move
clewellyn-nava 4ba103c
Now it builds
clewellyn-nava c290776
Now it builds (final)
clewellyn-nava 8ea8d98
Remove unnecessary dependency
clewellyn-nava be4ec21
Added entity concept + suppermapped classes they extend
clewellyn-nava ebf84e4
Initial work on institutional
clewellyn-nava f66b8a1
Create framework for profiles
clewellyn-nava b3d9dd3
Fix JPA queries
clewellyn-nava f5ad0bb
Merge branch 'master' into clewellyn-nava/BFD-4693/basis-profile-fram…
clewellyn-nava baea40a
Add table checks to script, add uv support
clewellyn-nava ba12b11
Add generated annotation to temporarily fool code coverage
clewellyn-nava bf185f8
Merge branch 'master' into clewellyn-nava/BFD-4693/basis-profile-fram…
clewellyn-nava be70731
Reverting unnecessary changes
clewellyn-nava 7690a2f
Merge branch 'master' into clewellyn-nava/BFD-4693/basis-profile-fram…
clewellyn-nava eef59ce
Merge branch 'master' into clewellyn-nava/BFD-4693/basis-profile-fram…
clewellyn-nava 9770fad
Fixing merge conflicts
clewellyn-nava 5bfb271
Update apps/bfd-server-ng/scripts/jpa_graph.py
clewellyn-nava b9b7dd4
Address github comments.
clewellyn-nava 33e8271
Merge branch 'master' into clewellyn-nava/BFD-4693/basis-profile-fram…
clewellyn-nava 467f94f
Update jpa_graph to latest version
clewellyn-nava b5c7a0b
Merge branch 'master' into clewellyn-nava/BFD-4693/basis-profile-fram…
clewellyn-nava 119a5eb
Update jpa_script and remove symlinks
clewellyn-nava c9164f3
Update apps/bfd-server-ng/scripts/README.md
clewellyn-nava 55b188c
remove hallucinations
clewellyn-nava 5c9b829
Merge branch 'master' into clewellyn-nava/BFD-4693/basis-profile-fram…
clewellyn-nava File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| 3.14 |
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| # bfd-server-ng Scripts | ||
|
|
||
| This folder contains scripts used in the maintenance of bfd-server-ng. At the moment, there is only one, but as codegen | ||
| becomes more sophisticated, it is possible for them to be moved over. | ||
|
|
||
| ## jpa_graph.py | ||
|
|
||
| Because we don't have IntelliJ pro, we have to write our own JPA mapping scripts. This script is a self contained uv script | ||
| that runs via: | ||
|
|
||
| ```aiignore | ||
| uv run jpa_graph.py | ||
| ``` | ||
|
|
||
| At the moment, this script is still being updated, but the goal of this script is to read in the mapping YAML files for | ||
| source columns and profiles, then create maps of the Java JPA objects (@Entity / @MappedSuperClass / @Embeddable) and | ||
| display the hierarchy in those files, and if they are compliant with the profiles defined in the YAML files. | ||
|
|
||
| This is done by matching the name of a profile in the @Entity class (CMS / Basis / Regular) and checking each @Column within | ||
| itself, its extension hierarchy, and all embedded objects recursively, and spitting out a rough analysis of their status. | ||
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.