Skip to content

changes based on Marc PRs for supporting complex ForwardRef#193

Closed
srivatsankrishnan wants to merge 4 commits intoNVIDIA-NeMo:parse_forwardreffrom
srivatsankrishnan:parse_forwardref
Closed

changes based on Marc PRs for supporting complex ForwardRef#193
srivatsankrishnan wants to merge 4 commits intoNVIDIA-NeMo:parse_forwardreffrom
srivatsankrishnan:parse_forwardref

Conversation

@srivatsankrishnan
Copy link

This is based on the original implementation by Marc for issues relating to passing the null_tokenizer and PackedSequence using Nemo Run CLI.

@run.cli.factory(target=MockDataModule, target_arg="tokenizer")
@run.autoconvert
def null_tokenizer()-> run.Config[AutoTokenizer]:
    return run.Config(
        get_nmt_tokenizer, library="null", model_name="NullTokenizer", vocab_size=256000
    )

Currently modified and the unit test seems to work. Will test on real use needed for me.

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