Skip to content

Conversation

josejuanmartinez
Copy link
Contributor

Description

Adds md models to Models Hub for 3.3.2 Healthcare release

Motivation and Context

Healthcare is releasing 3.3.2, and several models have been uploaded to the Hub. For these new models, we need to merge the md files.

How Has This Been Tested?

Checking models are downloadable and documentation looks ok.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Code improvements with no or little impact
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING page.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

murat-gunay and others added 30 commits November 3, 2021 13:40
Fixes the reference to the model in the python code section. Also adds a missing "clinical/models" path for properly retrieving the model from S3
Adds missing quote in python and scala code sections
After moving roberta embeddings to public, the code examples should change
Adds the 'es' language tag to pretrained in scala code
The ER header was 'icd10_code' when it should be 'snomed_code'
Adds more indentation to benchmarking section
Fixing indentation in results section
`roberta_base_biomedical` is not part of Healthcare suit, it's opensource, so no "clinical/models" reference required.
There was an incorrect quote in a word. This commit fixes it.
Removes extra quotes in some code examples (modelshub already add the format quotes automatically)
Adds a simplified version of the pipeline reusing several components

## Data Source

[https://huggingface.co/aychang/bert-base-cased-trec-coarse](https://huggingface.co/aychang/bert-base-cased-trec-coarse)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There are metrics available:
image


## Data Source

[https://huggingface.co/Hate-speech-CNERG/bert-base-uncased-hatexplain](https://huggingface.co/Hate-speech-CNERG/bert-base-uncased-hatexplain)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There are metrics here:
image

Use AUCROC BERT-HateXplain [LIME]


## Data Source

[https://huggingface.co/mrm8488/bert-tiny-finetuned-sms-spam-detection](https://huggingface.co/mrm8488/bert-tiny-finetuned-sms-spam-detection)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We have metrics in HF:
image


pipeline = Pipeline(stages=[document_assembler, tokenizer, sequenceClassifier])

example = spark.createDataFrame([['Because you need me Every single day Trying to find me But you don't know why Trying to find me again But you don't know how Trying to find me again Every single day']]).toDF("text")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This won't work, conflicting ' . Start and finish with "


## Data Source

[https://huggingface.co/juliensimon/autonlp-song-lyrics-18753417](https://huggingface.co/juliensimon/autonlp-song-lyrics-18753417)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We have metrics:
image

@JohnSnowLabs JohnSnowLabs deleted a comment from murat-gunay Nov 9, 2021
Copy link
Contributor Author

@josejuanmartinez josejuanmartinez left a comment

Choose a reason for hiding this comment

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

Now it seems much better. Thank you @gadde5300 !

@vkocaman vkocaman merged commit e6ce8df into master Nov 9, 2021
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.

8 participants