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

docs: emphasize LLMs and re-usable code in NLP docs #279

Merged
merged 1 commit into from
Oct 9, 2023

Conversation

mikix
Copy link
Contributor

@mikix mikix commented Oct 9, 2023

Checklist

  • Consider if documentation (like in docs/) needs to be updated
  • Consider if tests should be added

## NLP Is Always Specific to a Study
## NLP Is Always Specific to a Clinical Purpose
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Andy bumped on this, so I'm softening this language a bit


{: .note }
Example: The `covid_symptom` study uses cTAKES and a negation transformer working together to tag
COVID-19 symptoms in clinical notes.
But another study might use the Llama2 large language model,
with a prompt like "Does this patient have a nosebleed?"

### But With Re-Usable Code
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hit them over the head that ETL is not out here hardcoding stuff

## Available NLP Strategies

### Large Language Models (LLMs)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved this section above cTAKES and talk more about local & cloud LLMs in general, to emphasize our support here.

Unfortunately, we don't actually have shipped code for a cloud LLM or this section could be stronger. But we can update it when we do.

@mikix mikix merged commit 2966ea8 into main Oct 9, 2023
2 checks passed
@mikix mikix deleted the mikix/nlp-doc-tweaks branch October 9, 2023 20:25
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.

2 participants