{"action":"created","issue":{"url":"https://api.github.com/repos/OpenNMT/OpenNMT/issues/24","repository_url":"https://api.github.com/repos/OpenNMT/OpenNMT","labels_url":"https://api.github.com/repos/OpenNMT/OpenNMT/issues/24/labels{/name}","comments_url":"https://api.github.com/repos/OpenNMT/OpenNMT/issues/24/comments","events_url":"https://api.github.com/repos/OpenNMT/OpenNMT/issues/24/events","html_url":"https://github.com/OpenNMT/OpenNMT/issues/24","id":197161404,"number":24,"title":"About character-aware model","user":{"login":"helson73","id":4516503,"avatar_url":"https://avatars.githubusercontent.com/u/4516503?v=3","gravatar_id":"","url":"https://api.github.com/users/helson73","html_url":"https://github.com/helson73","followers_url":"https://api.github.com/users/helson73/followers","following_url":"https://api.github.com/users/helson73/following{/other_user}","gists_url":"https://api.github.com/users/helson73/gists{/gist_id}","starred_url":"https://api.github.com/users/helson73/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/helson73/subscriptions","organizations_url":"https://api.github.com/users/helson73/orgs","repos_url":"https://api.github.com/users/helson73/repos","events_url":"https://api.github.com/users/helson73/events{/privacy}","received_events_url":"https://api.github.com/users/helson73/received_events","type":"User","site_admin":false},"labels":[{"id":467891092,"url":"https://api.github.com/repos/OpenNMT/OpenNMT/labels/enhancement","name":"enhancement","color":"84b6eb","default":true}],"state":"open","locked":false,"assignee":{"login":"yoonkim","id":5378003,"avatar_url":"https://avatars.githubusercontent.com/u/5378003?v=3","gravatar_id":"","url":"https://api.github.com/users/yoonkim","html_url":"https://github.com/yoonkim","followers_url":"https://api.github.com/users/yoonkim/followers","following_url":"https://api.github.com/users/yoonkim/following{/other_user}","gists_url":"https://api.github.com/users/yoonkim/gists{/gist_id}","starred_url":"https://api.github.com/users/yoonkim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yoonkim/subscriptions","organizations_url":"https://api.github.com/users/yoonkim/orgs","repos_url":"https://api.github.com/users/yoonkim/repos","events_url":"https://api.github.com/users/yoonkim/events{/privacy}","received_events_url":"https://api.github.com/users/yoonkim/received_events","type":"User","site_admin":false},"assignees":[{"login":"yoonkim","id":5378003,"avatar_url":"https://avatars.githubusercontent.com/u/5378003?v=3","gravatar_id":"","url":"https://api.github.com/users/yoonkim","html_url":"https://github.com/yoonkim","followers_url":"https://api.github.com/users/yoonkim/followers","following_url":"https://api.github.com/users/yoonkim/following{/other_user}","gists_url":"https://api.github.com/users/yoonkim/gists{/gist_id}","starred_url":"https://api.github.com/users/yoonkim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yoonkim/subscriptions","organizations_url":"https://api.github.com/users/yoonkim/orgs","repos_url":"https://api.github.com/users/yoonkim/repos","events_url":"https://api.github.com/users/yoonkim/events{/privacy}","received_events_url":"https://api.github.com/users/yoonkim/received_events","type":"User","site_admin":false}],"milestone":null,"comments":3,"created_at":"2016-12-22T12:10:45Z","updated_at":"2016-12-22T20:38:43Z","closed_at":null,"body":"Hi, thanks for releasing such a beautiful toolkit !\r\nThis library seems like follow https://github.com/harvardnlp/seq2seq-attn,\r\nhowever compare to seq2seq-attn, cnn-based character-aware encoder is not available.\r\nIs there any plan to add such feature(cnn) in the future?"},"comment":{"url":"https://api.github.com/repos/OpenNMT/OpenNMT/issues/comments/268885667","html_url":"https://github.com/OpenNMT/OpenNMT/issues/24#issuecomment-268885667","issue_url":"https://api.github.com/repos/OpenNMT/OpenNMT/issues/24","id":268885667,"user":{"login":"helson73","id":4516503,"avatar_url":"https://avatars.githubusercontent.com/u/4516503?v=3","gravatar_id":"","url":"https://api.github.com/users/helson73","html_url":"https://github.com/helson73","followers_url":"https://api.github.com/users/helson73/followers","following_url":"https://api.github.com/users/helson73/following{/other_user}","gists_url":"https://api.github.com/users/helson73/gists{/gist_id}","starred_url":"https://api.github.com/users/helson73/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/helson73/subscriptions","organizations_url":"https://api.github.com/users/helson73/orgs","repos_url":"https://api.github.com/users/helson73/repos","events_url":"https://api.github.com/users/helson73/events{/privacy}","received_events_url":"https://api.github.com/users/helson73/received_events","type":"User","site_admin":false},"created_at":"2016-12-22T20:38:43Z","updated_at":"2016-12-22T20:38:43Z","body":"@srush Thanks, unfortunately in my language pair, applying BPE is not possible. \r\nI just implemented it myself, and also advanced Bayesian dropouts,\r\n`https://arxiv.org/abs/1512.05287`, and still tuning. \r\nI am wondering why don't you guys add this version of dropout? Current dropout is too naive.\r\nI found it can be implemented in a very simple way. Just rewrite nn.Dropout to force noise be fixed during all time steps, and tie noise tensors between clones, and also edit LSTM.lua to apply dropout to all nodes which feed to nn.Linear Nodes. I will share the result when the exp is complete.\r\n\r\n"}}
{"actor":{"display_login":"helson73"}}