allenai / allenai/vampire

Incorrect type annotation for encoder aggregation

オープン
#67 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Python
スター
174
フォーク
33
PR マージ指標
30日以内にマージされた PR はありません

説明

The constructor of Seq2Seq defines aggregations as a `str`, but the forward method treats it as a sequence, and the jsonnet file splits from a comma-separated string to an array. I'm not sure if something has changed in allennlp, but for me, this results in an exception that it's expecting aggregations to be a string.

https://github.com/allenai/vampire/blob/cf5c2fe17caf73bbd0a470c48cba4d3ab4e847ff/vampire/modules/encoder.py#L76

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。