Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

Commit

Permalink
Remove duplicate TextField (#344)
Browse files Browse the repository at this point in the history
* Remove duplicate TextField

* Update CHANGELOG

* Quotes

Co-authored-by: Dirk Groeneveld <dirkg@allenai.org>
  • Loading branch information
MaksymDel and dirkgr committed Aug 24, 2022
1 parent bac6505 commit 3e3b3ec
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## Unreleased

### Fixed

- Fixed redundant `TextField` wrapping in `TransformerSuperGlueRteReader`.

## [v2.10.0](https://github.com/allenai/allennlp-models/releases/tag/v2.10.0) - 2022-07-14

### Added
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ def text_to_instance(
premise_and_hypothesis = TextField(
self._tokenizer.add_special_tokens(tokenized_premise, tokenized_hypothesis),
)
fields["tokens"] = TextField(premise_and_hypothesis)
fields["tokens"] = premise_and_hypothesis

# make the metadata
metadata = {
Expand Down

0 comments on commit 3e3b3ec

Please sign in to comment.