Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add distributed implementation for understand sentiment LSTM model #7809

Closed
wants to merge 6 commits into from

Conversation

abhinavarora
Copy link
Contributor

Fixes #7749

@abhinavarora abhinavarora self-assigned this Jan 24, 2018
putcn
putcn previously requested changes Jan 24, 2018
Copy link
Contributor

@putcn putcn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

error while running this test:

Traceback (most recent call last):
  File "notest_understand_sentiment_lstm_dist.py", line 187, in <module>
    main()
  File "notest_understand_sentiment_lstm_dist.py", line 131, in main
    dict_dim=dict_dim, class_dim=class_dim)
  File "notest_understand_sentiment_lstm_dist.py", line 73, in lstm_net
    emb = fluid.layers.transpose(x=emb, axis=[1, 0, 2])
TypeError: transpose() got an unexpected keyword argument 'axis'

@abhinavarora abhinavarora dismissed putcn’s stale review January 24, 2018 19:30

Added commits to fix the issue

@abhinavarora
Copy link
Contributor Author

@putcn Thank you for pointing that out. It seems the method signature was updated recently. I have fixed the issue in commit aaaf995

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants