Fix calculation error when ref is empty

[voidful]

This pull request fixing the issue that any of the ref is empty

When the number of ref is inconsistent, i will fill a empty string as padding. which causing an error.

scores = n.compute_metrics(ref_list=[
            [
                "this is one reference sentence for sentence1",
                ""
            ],
            [
                "this is one more reference sentence for sentence1",
                "this is the second reference sentence for sentence2"
            ],
        ],
            hyp_list=[
                "this is the model generated sentence1 which seems good enough",
                "this is sentence2 which has been generated by your model"
            ]
        )

[msftclas]

All CLA requirements met.

[juharris]

Thanks for pointing this out. The references are the targets that the generated hypothesis should match. It's possible that a target would indeed be an empty string so I think we should correct what is actually causing the error instead of silently ignoring empty strings which could mean that a hypothesis is compared with a target that it wasn't meant to be compared with. For example:

References:

1. "Sentence 1"
2. "" (this one would get filtered out)
3. "Sentence 3"

Hypotheses:

1. "Sentence 1"
2. "Sentence 2"
3. "Sentence 3"

[voidful]

I agree that we should correct what is actually causing the error
To clear this in more general way:
when one of the ref is empty or hyp is empty

ref=["this is a test",""],
hyp="this is a good test"

ref=["this is a good test"],
hyp=""

vectorize metric(Skip-thought/ glove_metrics) will cause error due to empty input to encode
so the following commit will try to correct it.

[temporaer]

Can you remove changes to your gitignore? They seem rather specific to your setup.

[voidful]

Ok I can remove it.

[juharris]

BTW You can also create a global gitignore: https://docs.github.com/en/github/getting-started-with-github/ignoring-files#configuring-ignored-files-for-all-repositories-on-your-computer

[voidful]

@juharris Thank you for your advice. It is what I need.

[temporaer]

Can you assert something here, so that tests can detect regressions?

[voidful]

I found that this case is not necessary. I will remove it

[temporaer]

Same, better to assert the result you expect.

[voidful]

I found that this case is not necessary. I will remove it