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

Feat refactor transformer #7

Merged
merged 5 commits into from Nov 27, 2020
Merged

Feat refactor transformer #7

merged 5 commits into from Nov 27, 2020

Conversation

robin1001
Copy link
Collaborator

No description provided.

Change-Id: Ib96eb2705df4a11e7bcba6d4cddb8dbbf1bc5fee
Change-Id: I6424043d1fc7df713f44b7702811ba30292c45c7
wenet/transformer/encoder.py Outdated Show resolved Hide resolved
wenet/transformer/encoder.py Outdated Show resolved Hide resolved
Change-Id: I9e33f81fade97bb6de308a378e3abb47cf68d5b3
Change-Id: I4331576084d533c4719d46128a4143576ff4d832
Copy link
Collaborator

@placebokkk placebokkk left a comment

Choose a reason for hiding this comment

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

please fix the typo

wenet/transformer/convolution.py Outdated Show resolved Hide resolved
odim: dimension of outputs
encoder_output_size: number of encoder projection units
dropout_rate: dropout rate (0.0 ~ 1.0)
reduce: reduce the CTC loss into a scalar
Copy link
Collaborator

Choose a reason for hiding this comment

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

if the reduction is always needed, this arg should be removed.
Or add a reduction_type arg for choosing sum or mean reduction.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

change it in the future if we have stable version

wenet/transformer/embedding.py Outdated Show resolved Hide resolved
wenet/transformer/encoder.py Outdated Show resolved Hide resolved
wenet/transformer/subsampling.py Outdated Show resolved Hide resolved
Change-Id: Ifc7a02dd6c5ebb189002e918cec929ac1b2560a9
@jschenxiaoyu jschenxiaoyu merged commit 4f034bb into main Nov 27, 2020
@jschenxiaoyu jschenxiaoyu deleted the feat-refactor-transformer branch November 27, 2020 04:28
robin1001 added a commit that referenced this pull request Jan 13, 2021
* [bugfix] fix optional parameter in run.sh and indent bug in ctc

Change-Id: Ib96eb2705df4a11e7bcba6d4cddb8dbbf1bc5fee

* [network] refactor transformer/conformer encoder

Change-Id: I6424043d1fc7df713f44b7702811ba30292c45c7

* [network] refine import order

Change-Id: I9e33f81fade97bb6de308a378e3abb47cf68d5b3

* [network] change Conv2dSubsampling to Conv2dSubsampling4

Change-Id: I4331576084d533c4719d46128a4143576ff4d832

* [network] fix topo and comment

Change-Id: Ifc7a02dd6c5ebb189002e918cec929ac1b2560a9
This was referenced Jul 27, 2021
wantt referenced this pull request Apr 13, 2022
* [runtime] refactor decoder, asr_model to support more platforms

* fix topo error when rescoring

* support Copy for asr_model

* fix wrong check

* refine copy

* set inter-op thread
@zjc6666 zjc6666 mentioned this pull request Aug 8, 2022
@bboyxu5928 bboyxu5928 mentioned this pull request Sep 3, 2022
@jangjun21 jangjun21 mentioned this pull request Oct 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants