Skip to content

Updates to the transformers conf docs and yaml file - #1467

Merged
omri374 merged 3 commits into
mainfrom
docs/refresh_transformers_docs
Oct 13, 2024
Merged

Updates to the transformers conf docs and yaml file#1467
omri374 merged 3 commits into
mainfrom
docs/refresh_transformers_docs

Conversation

@omri374

@omri374 omri374 commented Oct 10, 2024

Copy link
Copy Markdown
Collaborator

updated the docs for a transformers based pipeline, to make it clearer, and added an example for configuration through code. Based on the discussion in #1463
Also changed the default config in the docs to match the one in the code.

@omri374 omri374 changed the title Small updates to the transformers conf docs and yaml file Updates to the transformers conf docs and yaml file Oct 10, 2024
@omri374
omri374 marked this pull request as ready for review October 10, 2024 07:00
@omri374
omri374 requested a review from michhar October 10, 2024 07:02
@omri374

omri374 commented Oct 10, 2024

Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@omri374
omri374 merged commit 21361f9 into main Oct 13, 2024
@omri374
omri374 deleted the docs/refresh_transformers_docs branch October 13, 2024 10:39
@michhar

michhar commented Oct 15, 2024

Copy link
Copy Markdown

@omri374 Thank you for making these changes. One question in my mind still is how to find the Hugging Face NER model labels...might be out of scope for the docs here, but perhaps a link or pointer to this could help since it's needed for the configuration file. Again, thanks for making things clearer.

@omri374

omri374 commented Oct 16, 2024

Copy link
Copy Markdown
Collaborator Author

Thanks @michhar. Will directing the user to look into the model's config (e.g. https://huggingface.co/dslim/bert-base-NER-uncased/blob/main/config.json) be the right approach? Not all models explicitly state the classes on their huggingface hub website.

@omri374
omri374 restored the docs/refresh_transformers_docs branch October 16, 2024 16:34
@omri374

omri374 commented Oct 16, 2024

Copy link
Copy Markdown
Collaborator Author

Please see #1472

prokopidis pushed a commit to prokopidis/presidio that referenced this pull request Jun 23, 2026
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.

3 participants