Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Fix the bug of BidirectionalCell #13575

Merged
merged 8 commits into from
Dec 13, 2018
Merged

Fix the bug of BidirectionalCell #13575

merged 8 commits into from
Dec 13, 2018

Commits on Dec 7, 2018

  1. Fix the bug of BidirectionalCell

    I did hybridize( ) and pass "valid_length" to the unroll( ) function of BidirectionalCell, then returned AssertionError in line 79. Because symbol.split( ) return a symbol but not a symbol list. Result in the length of inputs dont equal parameter "length"  when call unroll( )  to compute r_outputs and r_states.
    BeyonderXX authored Dec 7, 2018
    Configuration menu
    Copy the full SHA
    d2aba22 View commit details
    Browse the repository at this point in the history

Commits on Dec 9, 2018

  1. Configuration menu
    Copy the full SHA
    0af9aaf View commit details
    Browse the repository at this point in the history
  2. Fix the bug of BidirectionalCell

    I did hybridize( ) and pass "valid_length" to the unroll( ) function of BidirectionalCell, then returned AssertionError in line 79. Because symbol.split( ) return a symbol but not a symbol list. Result in the length of inputs dont equal parameter "length"  when call unroll( )  to compute r_outputs and r_states.
    BeyonderXX committed Dec 9, 2018
    Configuration menu
    Copy the full SHA
    1a3ff5d View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    0213015 View commit details
    Browse the repository at this point in the history

Commits on Dec 10, 2018

  1. fix test_bidirectional_unroll_valid_length( )

    Fix the error of parameter.
    BeyonderXX authored Dec 10, 2018
    Configuration menu
    Copy the full SHA
    14157e6 View commit details
    Browse the repository at this point in the history
  2. Fix the bug of BidirectionalCell

    I did hybridize( ) and pass "valid_length" to the unroll( ) function of BidirectionalCell, then returned AssertionError in line 79. Because symbol.split( ) return a symbol but not a symbol list. Result in the length of inputs dont equal parameter "length"  when call unroll( )  to compute r_outputs and r_states.
    BeyonderXX authored and xiaowang11 committed Dec 10, 2018
    Configuration menu
    Copy the full SHA
    c2dd9d4 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    7710970 View commit details
    Browse the repository at this point in the history

Commits on Dec 12, 2018

  1. fix test_bidirectional_unroll_valid_length( )

    BeyonderXX authored and xiaowang11 committed Dec 12, 2018
    Configuration menu
    Copy the full SHA
    2f412e5 View commit details
    Browse the repository at this point in the history