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

Adding carry-over status and carry-over values to SGD model #736

Merged
merged 101 commits into from
Jun 17, 2020

Conversation

VahidooX
Copy link
Collaborator

No description provided.

VahidooX added 30 commits May 18, 2020 14:22
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
VahidooX added 17 commits June 16, 2020 19:05
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
…er_carryover

Signed-off-by: Vahid Noroozi <[email protected]>

# Conflicts:
#	docs/sources/source/nlp/dialogue_state_tracking.rst
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
Signed-off-by: Vahid Noroozi <[email protected]>
self.noncategorical_slot_status[slot_idx] = STATUS_CARRY
else:
self.noncategorical_slot_status[slot_idx] = STATUS_ACTIVE

Copy link
Collaborator

Choose a reason for hiding this comment

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

Please move self.noncategorical_slot_status[slot_idx] = STATUS_ACTIVE back directly after else, the change happens only when self._add_carry_status is True

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

@VahidooX VahidooX requested a review from ekmb June 17, 2020 18:43
ekmb
ekmb previously approved these changes Jun 17, 2020
Signed-off-by: Vahid Noroozi <[email protected]>
@yzhang123 yzhang123 merged commit 3ce43fb into NVIDIA:master Jun 17, 2020
@VahidooX VahidooX deleted the sgd_master_carryover branch December 11, 2021 00:26
dcurran90 pushed a commit to dcurran90/NeMo that referenced this pull request Oct 15, 2024
* Use setuptools_scm to auto-generate version

[setuptools_scm](https://github.com/pypa/setuptools_scm) is the de-facto
standard to automate the version information of a Python project. It
uses information from version control system like git to generate
[PEP 440](https://peps.python.org/pep-0440/) version identifiers.

The version number is generated on build time and included in source
dist and Python wheels. It works with `pip install` from a local
checkout or git checkout URL.

For git repositories, the version from the last git tag in the history.
If the current checkout is not in tag, setuptools_scm includes git tag
revision and count since last tag in the version. This information is
useful to debug development checkouts.

Example: `cli-0.10.2.dev4+g724d895` means four commits (`dev4`) after
release tag `v0.10.1` (sic!).

The change also fixes a problem with PR NVIDIA#727. It added dynamic version
without actually making the version dynamic.

Also run tests whenever `pyproject.toml` or requirements are modified.

Resolves: NVIDIA#268
Resolves: NVIDIA#736
Signed-off-by: Christian Heimes <[email protected]>
Signed-off-by: Mark Sturdevant <[email protected]>
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.

3 participants