noamaanMulla-03/transformers-issues huggingface dataset:
All learning rates are 0 warning
Author: sajastu
Pull request: no

- `transformers` version: 3.0.2
- Platform: Linux
- Python version: 3.6
- PyTorch version (GPU?): 1.4 (GPU)
- Using GPU in script?: Yes
- Using distributed or parallel set-up in script?: No

### Who can help
@sshleifer 

## Information

Model I am using (Bert, XLNet ...): BART

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:
* [ ] an official GLUE/SQUaD task: (give the name)
* [x] my own task or dataset: (give details below)

## To reproduce

Steps to reproduce the behavior:

1. Running the example script in https://github.com/huggingface/transformers/tree/master/examples/seq2seq (finetune_bart_tiny.sh), I'm getting this warning in the beginning of training. However, the training process is continuing after that. 

Warning:
```
finetune.py:245: UserWarning: All learning rates are 0
  warnings.warn("All learning rates are 0")
Epoch 1:   0%|      
/home/sajad/anaconda3/lib/python3.6/site-packages/torch/optim/lr_scheduler.py:224: UserWarning: To get the last learning rate computed by the scheduler, please use `get_last_lr()`.
  warnings.warn("To get the last learning rate computed by the scheduler, "
```

## Expected behavior
While the training seemingly goes well, I'm wondering if this warning would cause problems, leading to deteriorate model's final performance? As a add-on, I've also incorporated the gradient checkpointing to some computational blocks of BART (modifying `modelling_bart.py` script a bit). But even w/o incorporating this module, I'm still getting this warning message? Any thoughts of how to solve it? 


Comments:
Duplicate of #5338 , ignore it.
Sorry for the confusion.

It makes sense what you answered at #5338, thanks for the clarification. I'm closing this issue!
fix huggingface/tokenizers#297 in 0.8.0
Author: sarahwie
Pull request: yes
Labels: name: wontfix


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

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

```diff
@@            Coverage Diff             @@
##           master    #4882      +/-   ##
==========================================
+ Coverage   76.46%   76.56%   +0.09%     
==========================================
  Files         128      128              
  Lines       21502    21502              
==========================================
+ Hits        16442    16462      +20     
+ Misses       5060     5040      -20     
```


| [Impacted Files](https://codecov.io/gh/huggingface/transformers/pull/4882?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [src/transformers/tokenization\_utils.py](https://codecov.io/gh/huggingface/transformers/pull/4882/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy90b2tlbml6YXRpb25fdXRpbHMucHk=) | `89.69% <0.00%> (ø)` | |
| [src/transformers/modeling\_tf\_utils.py](https://codecov.io/gh/huggingface/transformers/pull/4882/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl91dGlscy5weQ==) | `87.12% <0.00%> (-0.16%)` | :arrow_down: |
| [src/transformers/modeling\_roberta.py](https://codecov.io/gh/huggingface/transformers/pull/4882/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ19yb2JlcnRhLnB5) | `94.73% <0.00%> (+10.04%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/huggingface/transformers/pull/4882?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/4882?src=pr&el=footer). Last update [02e5f79...1ce1fd3](https://codecov.io/gh/huggingface/transformers/pull/4882?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-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.
Discriminative fine-tuning for new (added) words
Author: Aktsvigun
Pull request: no
Labels: name: wontfix

Good afternoon,
I have a question about the way of changing the learning rate for some parameters of the model. Let's say we have a BERT model and we have added a few new tokens. Consequently, we need to resize the embedding layer and initialize the embeddings for new words randomly. Meanwhile the learning rate of 3e-5 is used to train the model (otherwise we "overjump" the global minimum). If we use this LR for the embeddings of the new words as well, they will hardly change and thus will be close to random, thereby, the reasonable way is to change the learning rate only for their embeddings (as it is done, for instance, in ULMFiT). The question is: is there a simple way to do it in HuggingFace? Or probably are there some examples of doing it? Thanks in advance!

Comments:
Finally found the solution. The optimizer should look like this:
```
optim.SGD([
    {'params': model.base.parameters()},
    {'params': model.classifier.parameters(), 'lr': 1e-3}
], lr=1e-2, momentum=0.9))
```

However, this approach does not work when I need to specify another LR not for the whole layer, but only for a few weights from it. Hence, the issue is still open. @LysandreJik @patrickvonplaten will be very grateful if you could help.

Should a bit specify the question, I need to fine-tune the model (as proposed [here](https://github.com/huggingface/transformers/tree/master/examples/language-modeling)), specifying special learning rate for only the new word embeddings (_ideally_) or at least for the whole embedding matrix.

Hi @Aktsvigun, 

Thanks for your detailed question! I'm not super familiar with these kind of specifics for training, but I'm not sure that this is even possible in PyTorch. 

Also, did you try to fine-tune the model normally as well without setting a specific learning rate to only one parameter of a layer? I would expect normal fine-tuning to also work quite well since the gradient (independently of the lr) for the newly added weight will be quite high and thus change significantly even for the same learning rate for all parameters.

@patrickvonplaten thank you for the answer! 

I did try with a simple LR but what urged me to the question is the difference in the results of a language model, fine-tuned without adding new words and the one with changed vocab. I took a small dataset (_~ 25 000 sentences_) and fine-tuned both models with `lr = 3e-5` (pretty standard as I know) and `num_epochs = 10`. The model without new words had an eval loss (on the validation sample) of **2.55**, while the loss for the one with 250 new words (not much really when the vocab size equals 50250) equaled **2.98**. The words are not so popular among the dataset itself (only 13 of them belong to top-250 most frequent words), therefore such a great difference can be caused only by underfitting in my view.

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.
Aborted (core dumped) or Kernel dies
Author: yashwatwani
Pull request: no
Labels: name: wontfix

Whenever I am trying to import tranfomers my kernel dies off in jupyter notebook .
tranformer version - 2.8.0
python version -3.7.7


Comments:
Do you mind showing us the lines that crash? Do you have a reproducible example?

+1. @yashwatwani do you have resolved it? I have the same problem too.

If I install transformers 2.8.0, it will produce error:
```
[1]    11267 segmentation fault (core dumped)  PYTHONPATH=. python apps/absa/main.py
```

If I upgrade to the latest version 2.11.0, no error happens.

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.
Latest version of transformers available via conda-forge?
Author: brik3012
Pull request: no
Labels: name: wontfix

On conda forge the last version is the 2.1.1,
Why is not updated to the latest? Is it in plan to update the version? 
We work in a limited environment and we are forced to work with conda-forge channel.
Thanks 

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.
Fixes paths with spaces in seq2seq example
Author: KylePiira
Pull request: yes

Fixes https://github.com/huggingface/transformers/issues/6477

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

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

```diff
@@            Coverage Diff             @@
##           master    #6493      +/-   ##
==========================================
+ Coverage   80.38%   80.59%   +0.21%     
==========================================
  Files         156      156              
  Lines       28058    28058              
==========================================
+ Hits        22554    22613      +59     
+ Misses       5504     5445      -59     
```


| [Impacted Files](https://codecov.io/gh/huggingface/transformers/pull/6493?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [src/transformers/trainer.py](https://codecov.io/gh/huggingface/transformers/pull/6493/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy90cmFpbmVyLnB5) | `37.84% <0.00%> (ø)` | |
| [src/transformers/modeling\_tf\_flaubert.py](https://codecov.io/gh/huggingface/transformers/pull/6493/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl9mbGF1YmVydC5weQ==) | `24.53% <0.00%> (-63.81%)` | :arrow_down: |
| [src/transformers/pipelines.py](https://codecov.io/gh/huggingface/transformers/pull/6493/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9waXBlbGluZXMucHk=) | `79.69% <0.00%> (-0.26%)` | :arrow_down: |
| [src/transformers/modeling\_bert.py](https://codecov.io/gh/huggingface/transformers/pull/6493/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ19iZXJ0LnB5) | `88.42% <0.00%> (+0.16%)` | :arrow_up: |
| [src/transformers/generation\_tf\_utils.py](https://codecov.io/gh/huggingface/transformers/pull/6493/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9nZW5lcmF0aW9uX3RmX3V0aWxzLnB5) | `86.71% <0.00%> (+0.25%)` | :arrow_up: |
| [src/transformers/modeling\_tf\_bert.py](https://codecov.io/gh/huggingface/transformers/pull/6493/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl9iZXJ0LnB5) | `98.38% <0.00%> (+29.31%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/huggingface/transformers/pull/6493?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/6493?src=pr&el=footer). Last update [24107c2...057a225](https://codecov.io/gh/huggingface/transformers/pull/6493?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


Thanks!
finetune.py: error: unrecognized arguments
Author: KylePiira
Pull request: no

### Who can help

 examples/distillation: @VictorSanh
 examples/seq2seq: @sshleifer

## Information

Model I am using (Bert, XLNet ...):

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:
* [ ] an official GLUE/SQUaD task: (give the name)
* [X] my own task or dataset: (give details below)

## To reproduce

Steps to reproduce the behavior:

1. Try running seq2seq/finetune.sh with data_dir or output_dir with escaped spaces in it
2. You'll get a `finetune.py: error: unrecognized arguments`

This is bad because Google Drive mounts at `/content/drive/My Drive/` in Colab and thus the example scripts won't work if saving or reading from Drive.

I've created a [Colab Notebook](https://colab.research.google.com/drive/1N-8m9FC9GbAywVJZAgSBkLqe24SPRfl8?usp=sharing) with repro.

The fix I've found is to change:
```
python finetune.py \
    --learning_rate=3e-5 \
    --fp16 \
    --gpus 1 \
    --do_train \
    --do_predict \
    --n_val 1000 \
    --val_check_interval 0.1 \
    $@
```
to 
```
python finetune.py \
    --learning_rate=3e-5 \
    --fp16 \
    --gpus 1 \
    --do_train \
    --do_predict \
    --n_val 1000 \
    --val_check_interval 0.1 \
    "$@"
```


Comments:
enable invocation of run_ner.py and utils_ner.py in cython
Author: mgoldey
Pull request: yes
Labels: name: wontfix

Due to extant issue https://github.com/cython/cython/issues/2903, `run_ner.py` and `utils_ner.py` (among others, I imagine) cannot be invoked inside Cython. By manually adding in annotations, these changes work around the missing features in Cython 3.7 re: PEP557.
https://github.com/cython/cython/pull/3400 ought to eventually fix the underlying issue.

I'm offering code here that works around this behavior in case it would be helpful to others.

Comments:
# [Codecov](https://codecov.io/gh/huggingface/transformers/pull/4917?src=pr&el=h1) Report
> Merging [#4917](https://codecov.io/gh/huggingface/transformers/pull/4917?src=pr&el=desc) into [master](https://codecov.io/gh/huggingface/transformers/commit/ef2dcdccaa9a115aca44d81f31c6dc4d32bebb3f&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/4917/graphs/tree.svg?width=650&height=150&src=pr&token=9qOlN6Hb1c)](https://codecov.io/gh/huggingface/transformers/pull/4917?src=pr&el=tree)

```diff
@@           Coverage Diff           @@
##           master    #4917   +/-   ##
=======================================
  Coverage   77.12%   77.13%           
=======================================
  Files         128      128           
  Lines       21650    21650           
=======================================
+ Hits        16698    16700    +2     
+ Misses       4952     4950    -2     
```


| [Impacted Files](https://codecov.io/gh/huggingface/transformers/pull/4917?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [src/transformers/file\_utils.py](https://codecov.io/gh/huggingface/transformers/pull/4917/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9maWxlX3V0aWxzLnB5) | `73.09% <0.00%> (-0.41%)` | :arrow_down: |
| [src/transformers/modeling\_utils.py](https://codecov.io/gh/huggingface/transformers/pull/4917/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ191dGlscy5weQ==) | `90.61% <0.00%> (+0.11%)` | :arrow_up: |
| [src/transformers/modeling\_tf\_utils.py](https://codecov.io/gh/huggingface/transformers/pull/4917/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl91dGlscy5weQ==) | `87.57% <0.00%> (+0.31%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/huggingface/transformers/pull/4917?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/4917?src=pr&el=footer). Last update [ef2dcdc...74e1e66](https://codecov.io/gh/huggingface/transformers/pull/4917?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-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.
File is not found due to extension
Author: halidziya
Pull request: no
Labels: name: wontfix

Hi, 

In the configuration where the internet is not available the system searches the cache directory. However url_to_filename(url, etag) returns filename with extension in Windows. Thus this line is not able to find the file locations. One remedy could be  using filename.split('.')[0] instead of filename
      

https://github.com/huggingface/transformers/blob/466aa57a45bfb9fc47d4b75d22c02c34b4b4b0fc/src/transformers/file_utils.py#L404

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.
attempt to fix onnx test
Author: sshleifer
Pull request: yes


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

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

```diff
@@            Coverage Diff             @@
##           master    #6528      +/-   ##
==========================================
+ Coverage   80.38%   80.59%   +0.20%     
==========================================
  Files         156      156              
  Lines       28058    28058              
==========================================
+ Hits        22554    22612      +58     
+ Misses       5504     5446      -58     
```


| [Impacted Files](https://codecov.io/gh/huggingface/transformers/pull/6528?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [src/transformers/modeling\_tf\_flaubert.py](https://codecov.io/gh/huggingface/transformers/pull/6528/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl9mbGF1YmVydC5weQ==) | `24.53% <0.00%> (-63.81%)` | :arrow_down: |
| [src/transformers/pipelines.py](https://codecov.io/gh/huggingface/transformers/pull/6528/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9waXBlbGluZXMucHk=) | `79.69% <0.00%> (-0.26%)` | :arrow_down: |
| [src/transformers/modeling\_bert.py](https://codecov.io/gh/huggingface/transformers/pull/6528/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ19iZXJ0LnB5) | `88.42% <0.00%> (+0.16%)` | :arrow_up: |
| [src/transformers/modeling\_tf\_bert.py](https://codecov.io/gh/huggingface/transformers/pull/6528/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl9iZXJ0LnB5) | `98.38% <0.00%> (+29.31%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/huggingface/transformers/pull/6528?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/6528?src=pr&el=footer). Last update [72add6c...821c990](https://codecov.io/gh/huggingface/transformers/pull/6528?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
Update bert-base-portuguese-cased and bert-large-portuguese-cased cards
Author: fabiocapsouza
Pull request: yes
Labels: name: model card


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

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

```diff
@@            Coverage Diff             @@
##           master    #6527      +/-   ##
==========================================
- Coverage   80.24%   79.97%   -0.28%     
==========================================
  Files         149      149              
  Lines       27680    27680              
==========================================
- Hits        22211    22136      -75     
- Misses       5469     5544      +75     
```


| [Impacted Files](https://codecov.io/gh/huggingface/transformers/pull/6527?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [src/transformers/tokenization\_albert.py](https://codecov.io/gh/huggingface/transformers/pull/6527/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy90b2tlbml6YXRpb25fYWxiZXJ0LnB5) | `28.84% <0.00%> (-58.66%)` | :arrow_down: |
| [src/transformers/modeling\_tf\_distilbert.py](https://codecov.io/gh/huggingface/transformers/pull/6527/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl9kaXN0aWxiZXJ0LnB5) | `64.47% <0.00%> (-32.95%)` | :arrow_down: |
| [src/transformers/tokenization\_dpr.py](https://codecov.io/gh/huggingface/transformers/pull/6527/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy90b2tlbml6YXRpb25fZHByLnB5) | `57.65% <0.00%> (+4.50%)` | :arrow_up: |
| [src/transformers/tokenization\_ctrl.py](https://codecov.io/gh/huggingface/transformers/pull/6527/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy90b2tlbml6YXRpb25fY3RybC5weQ==) | `96.11% <0.00%> (+17.47%)` | :arrow_up: |
| [src/transformers/modeling\_tf\_flaubert.py](https://codecov.io/gh/huggingface/transformers/pull/6527/diff?src=pr&el=tree#diff-c3JjL3RyYW5zZm9ybWVycy9tb2RlbGluZ190Zl9mbGF1YmVydC5weQ==) | `87.73% <0.00%> (+63.19%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/huggingface/transformers/pull/6527?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/6527?src=pr&el=footer). Last update [00bb0b2...1d585e3](https://codecov.io/gh/huggingface/transformers/pull/6527?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).