Logger bug fix

[agv7]

What does this PR do?

Solves the "module 'textattack' has no attribute 'shared'" error that comes up whenever doing "from textattack.augmentation import EmbeddingAugmenter".

Before Change:

After Change:

Summary

Earlier, there was "logger = textattack.shared.utils.logger", but according to https://stackoverflow.com/questions/8899198/module-has-no-attribute and the current way that attacker.py uses logger (https://github.com/QData/TextAttack/blob/907ec46556d8c36bfe62b4db0484dfcbd9b706db/textattack/attacker.py), it looks like the best way to solve the problem is to replace the line with a direct import. The problem earlier was that there was no direct way to get the attribute logger, but this PR should fix that problem.

Additions

• The line "from textattack.shared.utils import logger" has been added.

Changes

• The line "logger = textattack.shared.utils.logger" has been changed to "from textattack.shared.utils import logger".

Deletions

• The line "logger = textattack.shared.utils.logger" has been deleted.

Checklist

• The title of your pull request should be a summary of its contribution.
• Please write detailed description of what parts have been newly added and what parts have been modified. Please also explain why certain changes were made.
• If your pull request addresses an issue, please mention the issue number in the pull request description to make sure they are linked (and people consulting the issue know you are working on it)
• To indicate a work in progress please mark it as a draft on Github.
• Make sure existing tests pass.
• Add relevant tests. No quality testing = no merge.
• All public methods must have informative docstrings that work nicely with sphinx. For new modules/files, please add/modify the appropriate .rst file in TextAttack/docs/apidoc.'

[Hanyu-Liu-123]

should we still initiate the logger, but this time with something like logger = logger?

[agv7]

I think the logger variable gets defined through the import itself, so it might not be needed.

[qiyanjun]

@Hanyu-Liu-123 is this ready to merge? The import makes sense!

[Hanyu-Liu-123]

Looks good to me! Thank you @ankitgv0