[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-12-04 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-441164957 I have resolved my doubt by initial the probility alphas with 'exp(x)' , rather than 'x'. this two method seem achieve the same .but the

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-12-04 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-441164957 I have resolved my doubt by initial the probility alphas with 'exp(x)' , rather than 'x'. this two method seem achieve the same .but the

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-12-01 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-441164957 I have resolved my doubt by initial the probility alphas with 'exp(x)' , rather than 'x'. the new result is numerical stability, this two

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-29 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-441164957 I have resolved my doubt by initial the begian probility alphas with 'exp(x)' , rather than 'x'. the new result is numerical stability, this

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-29 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-441164957 I have resolved my doubt by initial the begian probility alphas with 'x' , rather than 'exp(x)'. the new result is numerical stability, this

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-29 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-443061810 ``` def _forward_alg(self, feats): alphas = [[0.] * self.tagset_size]

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-29 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-441164957 I have resolved my doubt by initial the begian probility alphas with 'exp(x)' , rather than 'x'. the new result is numerical stability, this

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-29 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-441164957 I have resolved my doubt by initial the begian probility alphas with 'x' , rather than 'exp(x)'. the new result is numerical stability, this

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum" and the second one should be

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum" and the second one should be

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum" and the second one should be

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum" and the second one should be

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum" ,and the second one is "log_sum"

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum" in theory .but it may causs a

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum_exp" ,and the second one is

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-21 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum_exp" ,and the second one is

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-21 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) ,but I think the first "log_sum_exp" in the red pane may be replaced with "sum_exp" ,and the second one is

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-21 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) ,but I think the first "log_sum_exp" in the red pane should be replaced with "sum_exp"

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-21 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) ,but I think the first "logsumexp" may be replaced with "sumexp"

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-21 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) ,but I think the first "logsumexp" should be replaced with "sumexp"

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-20 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440134211 here are the codes https://github.com/apache/incubator-mxnet/blob/master/example/gluon/lstm_crf/lstm_crf.py

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-19 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440133690 Should the " log_sum_exp " in the red pane be replaced by " nd.sum "? this is my doubt