noamaanMulla-03/transformers-issues huggingface dataset:
[Docs] Incorrect links to models in the Summary of the Models page
Author: mikaelsouza
Pull request: no

### Issues with the Summary of the Models page
- Description: 
	- Clicking on any model documentation located in https://huggingface.co/transformers/model_summary.html leads us to an empty page.
- Possible Fix: 
	- I've noticed that the GPT link is https://huggingface.co/model_doc/gpt while it should be https://huggingface.co/transformers/model_doc/gpt.html
	- This fix seems to resolve all other issues in the page.

- Observations:
	- I've tried to clone this repo and fix this issue, but for some reason, when I build my own version of the documentation, it doesn't use the `.../transformers/...` path inside the url leading to a different issue from the current online documentation.

- Examples:
<img width="726" alt="Screen Shot 2020-07-06 at 17 28 50" src="https://user-images.githubusercontent.com/9092284/86648750-20481b80-bfaf-11ea-9d3e-b8656d220fdb.png">
EXPECTED:
<img width="1280" alt="Screen Shot 2020-07-06 at 17 29 17" src="https://user-images.githubusercontent.com/9092284/86648771-250ccf80-bfaf-11ea-8552-dd5ab52e36b9.png">
RESULT:
<img width="742" alt="Screen Shot 2020-07-06 at 17 29 01" src="https://user-images.githubusercontent.com/9092284/86648785-28a05680-bfaf-11ea-9578-e16407f58fb4.png">


Comments:
Fixed in #5566
Add DeeBERT (entropy-based early exiting for *BERT)
Author: ji-xin
Pull request: yes

Add DeeBERT (entropy-based early exiting for *BERT).
Paper: https://www.aclweb.org/anthology/2020.acl-main.204/
Based on its original repository: https://github.com/castorini/DeeBERT 

Comments:
# [Codecov](https://codecov.io/gh/huggingface/transformers/pull/5477?src=pr&el=h1) Report
> Merging [#5477](https://codecov.io/gh/huggingface/transformers/pull/5477?src=pr&el=desc) into [master](https://codecov.io/gh/huggingface/transformers/commit/58cca47c16149e43d1b516623d59e3c5d97f695e&el=desc) will **increase** coverage by `0.41%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/huggingface/transformers/pull/5477/graphs/tree.svg?width=650&height=150&src=pr&token=9qOlN6Hb1c)](https://codecov.io/gh/huggingface/transformers/pull/5477?src=pr&el=tree)

```diff
@@            Coverage Diff             @@
##           master    #5477      +/-   ##
==========================================
+ Coverage   77.83%   78.25%   +0.41%     
==========================================
  Files         141      141              
  Lines       24634    24634              
==========================================
+ Hits        19175    19278     +103     
+ Misses       5459     5356     -103     
```


| [Impacted Files](https://codecov.io/gh/huggingface/transformers/pull/5477?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [src/transformers/modeling\_tf\_roberta.py](https://codecov.io/gh/huggingface/transformers/pull/5477/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl9yb2JlcnRhLnB5) | `43.47% <0.00%> (-49.57%)` | :arrow_down: |
| [src/transformers/modeling\_openai.py](https://codecov.io/gh/huggingface/transformers/pull/5477/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ19vcGVuYWkucHk=) | `81.09% <0.00%> (+1.37%)` | :arrow_up: |
| [src/transformers/generation\_tf\_utils.py](https://codecov.io/gh/huggingface/transformers/pull/5477/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9nZW5lcmF0aW9uX3RmX3V0aWxzLnB5) | `86.71% <0.00%> (+1.50%)` | :arrow_up: |
| [src/transformers/modeling\_tf\_openai.py](https://codecov.io/gh/huggingface/transformers/pull/5477/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl9vcGVuYWkucHk=) | `94.98% <0.00%> (+74.19%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/huggingface/transformers/pull/5477?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/huggingface/transformers/pull/5477?src=pr&el=footer). Last update [58cca47...f44de41](https://codecov.io/gh/huggingface/transformers/pull/5477?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


Btw: would be awesome so see a token classification example 😅

Hi @JetRunner, thanks for the review! I have updated according to your suggestions.

2 checks fail, however they don't seem relevant to my commits.

@LysandreJik Thanks for the comments and I've updated accordingly!
In Tensorflow the serving is very slow
Author: only-yao
Pull request: no

# ❓ I'm using the gpt2 model, In Tensorflow the serving is very slow, Tensorflow is much faster.

## Details
I use tf.saved_model save model
`tf.saved_model.save(model, export_dir="/model/2/")`

Tensorflow reads the model
```
imported = tf.saved_model.load("/model/2/")
f = imported.signatures["serving_default"]
```

The speed of Tensorflow is much higher than that of tensorflow
In the same model without the use of transformers, tensorFlow and TensorFlow serve at the same speed

Comments:
I don't get it
How can I mute the info from BertTokenizerFast.encode_plus
Author: 131250208
Pull request: no
Labels: name: wontfix

# 🐛 Bug

## Information

Model I am using: 
Bert

Language I am using the model on (English, Chinese ...):
Chinese

The problem arises when using:
* [ ] the official example scripts: (give details below)
* [ ] my own modified scripts: (give details below)
```python
tokenizer.encode_plus(...)
```
Every time I run this line, the info "Fast tokenizers add special tokens by default. To remove special tokens, please specify`add_special_tokens=False` during the initialisation rather than when calling `encode`,`encode_plus` or `batch_encode_plus`." will come out.

Even though I have already set the add_special_tokens to False
```python
tokenizer = BertTokenizerFast.from_pretrained(model_path, add_special_tokens = False)
```
The info can be very annoying when I run an iteration loop. How can I mute this info.

## Environment info
     
- `transformers` version: 2.8.0
- Platform: Linux-4.4.0-87-generic-x86_64-with-debian-stretch-sid
- Python version: 3.6.9
- PyTorch version (GPU?): 1.4.0+cu100 (True)
- Tensorflow version (GPU?): 2.0.0 (True)
- Using GPU in script?: yes
- Using distributed or parallel set-up in script?: no



Comments:
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
[PL examples]: fix progress bar bug
Author: hugoabonizio
Pull request: yes
Labels: name: wontfix

As shown by @prabalbansal in #3576, the fine tuning script for seq2seq models is failing with the following error:

![](https://user-images.githubusercontent.com/30004110/78461843-b2006e00-76cc-11ea-9be8-06171cad86e5.png)

This PR includes the fix suggested by @sshleifer in [this comment](https://github.com/huggingface/transformers/issues/3576#issuecomment-611755174), which made it work, but it seems to be suboptimal since the loss is not shown in the progress.

Comments:
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
[readability] Consolidate prune_heads logic to PretrainedModel.
Author: sshleifer
Pull request: no
Labels: name: Help wanted

name: PyTorch

name: Good First Issue

Many models have identical implementations of `prune_heads` it would be nice to store that implementation as a method on `PretrainedModel` and reduce the redundancy.

Comments:
Hi @sshleifer , 
I am new to opensource and would like to help out with this issue. Could you please point me to some guide to setting up the project locally.
Thanks.

I got the contributing.md file. Thanks anyways. :)

Hi @sshleifer 
I am looking for a contribution. Is the issue is still open?
Thanks and Regards

@Shandilya21 It looks open. I was trying to work on it and got stuck at a point. Do let me know if you would like to discuss

Hi @noelmat Okay, can you tell me where you got stuck? I am happy to discuss this.

Is anybody working on this? I'm new to open source but I'd like to give it a shot

Go for it!

> Is anybody working on this? I'm new to open source but I'd like to give it a shot

Yeah, I am in.. I also wanna work on this issue.
Issue is to implement `prune_heads` as a method in `PretrainedModel`

@yugaljain1999 made some progress on this?

I think this is done. Happy to find new bugs if anyone is on the hunt!
TFTrainer does not consider number of epochs when calculating learning rate
Author: borisdayma
Pull request: no

# 🐛 Bug
`TFTrainer` does not consider number of epochs when calculating learning rate

## Information

When using `TFTrainer`, learning rate decreases to 0 at the end of the first epoch, even when we want to train on multiple epochs.

The problem arises when using:
* [X] the official example scripts: (give details below)
* [ ] my own modified scripts: (give details below)

The tasks I am working on is:
* [X] an official GLUE/SQUaD task: run_tf_glue with mrpc task
* [ ] my own task or dataset: (give details below)

## To reproduce
`python run_tf_glue.py --model_name_or_path bert-base-cased --task_name MRPC --do_train --do_eval --max_seq_length 128 --per_device_train_batch_size 32 --learning_rate 2e-5 --num_train_epochs 3.0 --output_dir /tmp/test_hf/ --overwrite_output_dir --logging_dir hf --evaluate_during_training --eval_steps 50 --logging_steps 10`


<!-- If you have code snippets, error messages, stack traces please provide them here as well.
     Important! Use code tags to correctly format your code. See https://help.github.com/en/github/writing-on-github/creating-and-highlighting-code-blocks#syntax-highlighting
     Do not use screenshots, as they are hard to read and (more importantly) don't allow others to copy-and-paste your code.-->

This is what we get, where learning rate is null after the first epoch:
![image](https://user-images.githubusercontent.com/715491/84722988-39c6ec00-af4a-11ea-9c80-f2213d5a579c.png)

You can refer to [W&B run](https://app.wandb.ai/borisd13/huggingface/runs/2zcsfumy?workspace=user-borisd13) for more details.

## Expected behavior

<!-- A clear and concise description of what you would expect to happen. -->

Learning rate should slowly decrease until end of 3rd epoch.

## Environment info
<!-- You can run the command `transformers-cli env` and copy-and-paste its output below.
     Don't forget to fill out the missing fields in that output! -->
     
- `transformers` version: 2.11.0
- Platform: Linux-5.3.0-53-generic-x86_64-with-glibc2.10
- Python version: 3.8.3
- PyTorch version (GPU?): N/A
- Tensorflow version (GPU?): 2.2.0 (True)
- Using GPU in script?: Yes (one only)
- Using distributed or parallel set-up in script?: No

@jplu I recorded the issue here so we don't forget to fix it.
Let me know if I can help.

Comments:
This is fixed
Add newly trained `calbert-tiny-uncased`
Author: txus
Pull request: yes
Labels: name: model card

Calbert is an open-source ALBERT trained on the Catalan OSCAR dataset. This is the `tiny` version, newly trained from a completely rewrite [in this repo](https://github.com/codegram/calbert), now using SentencePiece which makes everything work much better with pipelines and the default tooling, as demonstrated in the model card.

Comments:
# [Codecov](https://codecov.io/gh/huggingface/transformers/pull/5599?src=pr&el=h1) Report
> Merging [#5599](https://codecov.io/gh/huggingface/transformers/pull/5599?src=pr&el=desc) into [master](https://codecov.io/gh/huggingface/transformers/commit/f82a2a5e8e6827343322a4a9831924c5bb9bd2b2&el=desc) will **not change** coverage.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/huggingface/transformers/pull/5599/graphs/tree.svg?width=650&height=150&src=pr&token=9qOlN6Hb1c)](https://codecov.io/gh/huggingface/transformers/pull/5599?src=pr&el=tree)

```diff
@@           Coverage Diff           @@
##           master    #5599   +/-   ##
=======================================
  Coverage   76.69%   76.69%           
=======================================
  Files         145      145           
  Lines       25351    25351           
=======================================
  Hits        19444    19444           
  Misses       5907     5907           
```



------

[Continue to review full report at Codecov](https://codecov.io/gh/huggingface/transformers/pull/5599?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/huggingface/transformers/pull/5599?src=pr&el=footer). Last update [f82a2a5...0a4786e](https://codecov.io/gh/huggingface/transformers/pull/5599?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


looks great, thanks for sharing!
create model cards for qg models
Author: patil-suraj
Pull request: yes
Labels: name: model card

cc @julien-c , @danielduckworth

Comments:
# [Codecov](https://codecov.io/gh/huggingface/transformers/pull/5610?src=pr&el=h1) Report
> Merging [#5610](https://codecov.io/gh/huggingface/transformers/pull/5610?src=pr&el=desc) into [master](https://codecov.io/gh/huggingface/transformers/commit/40d98ebf50c4662bcd6dce6395bbed0b2142ea52&el=desc) will **increase** coverage by `1.23%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/huggingface/transformers/pull/5610/graphs/tree.svg?width=650&height=150&src=pr&token=9qOlN6Hb1c)](https://codecov.io/gh/huggingface/transformers/pull/5610?src=pr&el=tree)

```diff
@@            Coverage Diff             @@
##           master    #5610      +/-   ##
==========================================
+ Coverage   76.88%   78.11%   +1.23%     
==========================================
  Files         145      145              
  Lines       25351    25351              
==========================================
+ Hits        19491    19804     +313     
+ Misses       5860     5547     -313     
```


| [Impacted Files](https://codecov.io/gh/huggingface/transformers/pull/5610?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [src/transformers/modeling\_tf\_roberta.py](https://codecov.io/gh/huggingface/transformers/pull/5610/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl9yb2JlcnRhLnB5) | `43.98% <0.00%> (-49.38%)` | :arrow_down: |
| [src/transformers/file\_utils.py](https://codecov.io/gh/huggingface/transformers/pull/5610/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9maWxlX3V0aWxzLnB5) | `79.26% <0.00%> (-0.34%)` | :arrow_down: |
| [src/transformers/modeling\_tf\_mobilebert.py](https://codecov.io/gh/huggingface/transformers/pull/5610/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl9tb2JpbGViZXJ0LnB5) | `96.77% <0.00%> (+73.38%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/huggingface/transformers/pull/5610?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/huggingface/transformers/pull/5610?src=pr&el=footer). Last update [40d98eb...d2b586a](https://codecov.io/gh/huggingface/transformers/pull/5610?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


Looks great!

This is really excellent work @patil-suraj and thanks for the thorough documentation.
doc fixes
Author: stas00
Pull request: yes

a few minor doc improvements.

Comments:
# [Codecov](https://codecov.io/gh/huggingface/transformers/pull/5613?src=pr&el=h1) Report
> Merging [#5613](https://codecov.io/gh/huggingface/transformers/pull/5613?src=pr&el=desc) into [master](https://codecov.io/gh/huggingface/transformers/commit/7d0ef0042042899b67867a4e2962d8e97fb5c6f5&el=desc) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/huggingface/transformers/pull/5613/graphs/tree.svg?width=650&height=150&src=pr&token=9qOlN6Hb1c)](https://codecov.io/gh/huggingface/transformers/pull/5613?src=pr&el=tree)

```diff
@@           Coverage Diff           @@
##           master    #5613   +/-   ##
=======================================
  Coverage   76.88%   76.88%           
=======================================
  Files         145      145           
  Lines       25355    25355           
=======================================
+ Hits        19494    19495    +1     
+ Misses       5861     5860    -1     
```


| [Impacted Files](https://codecov.io/gh/huggingface/transformers/pull/5613?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [src/transformers/generation\_tf\_utils.py](https://codecov.io/gh/huggingface/transformers/pull/5613/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9nZW5lcmF0aW9uX3RmX3V0aWxzLnB5) | `86.46% <0.00%> (+0.25%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/huggingface/transformers/pull/5613?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/huggingface/transformers/pull/5613?src=pr&el=footer). Last update [7d0ef00...f052965](https://codecov.io/gh/huggingface/transformers/pull/5613?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
Add BPE dropout to tokenizers
Author: c00k1ez
Pull request: no
Labels: name: wontfix

# 🚀 Feature request

Hi! 
Thank you for the great library! 
Several days ago I was trying to fine-tune the GPT2 model with a small dataset. It was a bit hard because the model overfitted every time.
So, after it, I found a great paper with simple sub-word regularization, that outperformed classic BPE on text generation tasks (like NMT).
[Original paper](https://arxiv.org/pdf/1910.13267.pdf).
Unfortunately, I can't find any python implementation, only C++ [here](https://github.com/VKCOM/YouTokenToMe/blob/master/youtokentome/cpp/bpe.cpp).

Metrics from the paper:
![pic](https://drive.google.com/uc?export=view&id=1DPbGxppTBj0GG7nfrxmCx2CZxtyHap0v)

Comments:
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
Not able to reproduce same CoLA result as huggingface defualt
Author: leo-liuzy
Pull request: no
Labels: name: wontfix

# 🐛 Bug
When I run your official `run_glue.py` script on CoLA, the performance is significantly lower than the ones in your website. Mine: ~10, Yours: ~50
## Information

Model I am using (Bert, XLNet ...):
bert-base-uncased
Language I am using the model on (English, Chinese ...):
English

The problem arises when using:
* [x] the official example scripts: (give details below)
* [ ] my own modified scripts: (give details below)

The tasks I am working on is:
* [x] an official GLUE/SQUaD task: (give the name)
* [ ] my own task or dataset: (give details below)

## To reproduce

Steps to reproduce the behavior:

1. Run my provided command line
2.
3.
```
export GLUE_DIR=downstream_datasets/glue
export TASK_NAME=CoLA
SEED=43
export CUDA_VISIBLE_DEVICES=0

python run_glue.py \
  --model_type bert \
  --model_name_or_path bert-base-uncased \
  --task_name $TASK_NAME \
  --do_train \
  --save_steps 200 \
  --do_eval \
  --data_dir $GLUE_DIR/$TASK_NAME \
  --max_seq_length 128 \
  --per_gpu_train_batch_size 32 \
  --learning_rate 2e-5 \
  --num_train_epochs 30 \
  --seed $SEED \
  --output_dir tmp/"$TASK_NAME"_seed"$SEED"/
```
<!-- If you have code snippets, error messages, stack traces please provide them here as well.
     Important! Use code tags to correctly format your code. See https://help.github.com/en/github/writing-on-github/creating-and-highlighting-code-blocks#syntax-highlighting
     Do not use screenshots, as they are hard to read and (more importantly) don't allow others to copy-and-paste your code.-->

## Expected behavior

<!-- A clear and concise description of what you would expect to happen. -->

## Environment info
<!-- You can run the command `transformers-cli env` and copy-and-paste its output below.
     Don't forget to fill out the missing fields in that output! -->
     
- `transformers` version: 1.17.2
- Platform: CentOS 7.7 64-bit
- Python version: python 3.7
- PyTorch version (GPU?): 1.5.0+cuda9.2
- Tensorflow version (GPU?):
- Using GPU in script?: Yes
- Using distributed or parallel set-up in script?: No, single machine


Comments:
Do you have a link to a TensorBoard or any other experiment tracking for your training?

You can also add the `--evaluate_during_training` flag to see the evolution of the eval metric during training.

Have you tried with fewer epochs? (e.g ```--num_train_epochs 3.0``` as stated in the repo's example)

Also what you need to consider is seed hyper-parameter.


This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
Create README.md
Author: bashartalafha
Pull request: yes
Labels: name: model card


Comments:
# [Codecov](https://codecov.io/gh/huggingface/transformers/pull/5601?src=pr&el=h1) Report
> Merging [#5601](https://codecov.io/gh/huggingface/transformers/pull/5601?src=pr&el=desc) into [master](https://codecov.io/gh/huggingface/transformers/commit/f82a2a5e8e6827343322a4a9831924c5bb9bd2b2&el=desc) will **increase** coverage by `0.18%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/huggingface/transformers/pull/5601/graphs/tree.svg?width=650&height=150&src=pr&token=9qOlN6Hb1c)](https://codecov.io/gh/huggingface/transformers/pull/5601?src=pr&el=tree)

```diff
@@            Coverage Diff             @@
##           master    #5601      +/-   ##
==========================================
+ Coverage   76.69%   76.88%   +0.18%     
==========================================
  Files         145      145              
  Lines       25351    25351              
==========================================
+ Hits        19444    19490      +46     
+ Misses       5907     5861      -46     
```


| [Impacted Files](https://codecov.io/gh/huggingface/transformers/pull/5601?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [src/transformers/modeling\_tf\_distilbert.py](https://codecov.io/gh/huggingface/transformers/pull/5601/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl9kaXN0aWxiZXJ0LnB5) | `64.90% <0.00%> (-33.90%)` | :arrow_down: |
| [src/transformers/modeling\_tf\_electra.py](https://codecov.io/gh/huggingface/transformers/pull/5601/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl9lbGVjdHJhLnB5) | `95.53% <0.00%> (+69.51%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/huggingface/transformers/pull/5601?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/huggingface/transformers/pull/5601?src=pr&el=footer). Last update [f82a2a5...a3f37fa](https://codecov.io/gh/huggingface/transformers/pull/5601?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


Thanks! image link seems broken, feel free to update in a subsequent PR.