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

Named modules output #268

Merged
merged 3 commits into from
Jan 15, 2020
Merged

Named modules output #268

merged 3 commits into from
Jan 15, 2020

Conversation

stasbel
Copy link
Contributor

@stasbel stasbel commented Jan 15, 2020

There are two styles for defining DAGs with multi-output modules.
Old (also supported):

x, y, z = data_layer()
f = preprocessor(x, y)
loss = xe(f, z)

New:

data = data_layer()  # Data layer output types keys: ('x', 'y', 'z').
f = preprocessor(data.x, data.y)
loss = xe(f, data.z)

Should be a lot cleaner for modules with 5+ output tensors.

Copy link
Member

@okuchaiev okuchaiev left a comment

Choose a reason for hiding this comment

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

looks good. Thanks!

@okuchaiev okuchaiev merged commit 5467aba into master Jan 15, 2020
@blisc blisc mentioned this pull request Jan 16, 2020
@blisc blisc deleted the flow-naming branch May 28, 2020 22:10
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.

2 participants