Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Support nested NeMo models #5671
Support nested NeMo models #5671
Changes from 42 commits
21f9e79
a830aee
cd494d1
a19197c
f8519a4
3295d75
b114920
7b661d3
b722181
35ffb35
f7d42f4
b8e0f82
47e5e89
3d5e13d
49d2895
f775a5d
10549cf
7a5d83a
0136793
2347400
40c5437
b276cb9
20067cb
77bb11b
da7de90
b5960d5
7337fd1
5581f18
f6f84aa
d970b50
13ba08f
96ffd43
75c9622
d498a26
07dc349
050a57c
0cae024
0589c33
764f76f
5bf9b0e
d929d3f
0e68a46
80a6706
eb6ec43
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Restoration paths were tempdirs, do we recreate those tempdirs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
restoration_paths
are only paths to.nemo
checkpoints, since we don't use.nemo
files inside parent.nemo
file. Using.nemo
in.nemo
will still break the code and should be avoided.I changed it to a
set
to unpack each checkpoint only onceThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to add a test where we attempt to register a Nemo file. Maybe check and raise an error ?