2 Comments
User's avatar
JR$31's avatar

Very nice post, thanks!

Just a question:

```

class SimpleTransformer(nn.Module):

...

embedding += self.pos_encoder(embedding) # This line seems to add an extra embedding, right?

...

```

Expand full comment
Evan Mattson's avatar

Great catch. Yes, that's right. I've updated the post and the GitHub code with the fix. Appreciate you bringing this up.

Expand full comment