-
Notifications
You must be signed in to change notification settings - Fork 10
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
Lift to SpeziLLM #44
Lift to SpeziLLM #44
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #44 +/- ##
===========================================
+ Coverage 38.55% 50.50% +11.95%
===========================================
Files 30 19 -11
Lines 1362 711 -651
===========================================
- Hits 525 359 -166
+ Misses 837 352 -485
Continue to review full report in Codecov by Sentry.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for all the updates! I took a look at the changes and added some comments here and there about suggestion improvements.
Excited to see this moving up the hierarchy of projects and to see the LLM advancements showing up in LLM on FHIR; great job @philippzagar! 🚀
LLMonFHIR/FHIR Interpretation/FHIRMultipleResourceInterpreter.swift
Outdated
Show resolved
Hide resolved
@PSchmiedmayer This PR can be merged after StanfordSpezi/SpeziFHIR#17 is merged and the dependency within LLMonFHIR is updated to the tagged release 🚀 |
Lift to SpeziLLM
♻️ Current situation & Problem
Dependent on old version of SpeziLLM
⚙️ Release Notes
📚 Documentation
Proper docs included
✅ Testing
All tests passing
Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines: