google-research / google-research/t5x

__init__.py missing in decoder only gin folder

Open
#1,271 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
3k
Forks
338
PR merge metrics
No merged PRs in 30d

Description

`t5x/t5x/examples/decoder_only/models/__init__.py` should be added to avoid gin error coming from the following:

```
import gin
config = gin.config.parse_config_file("t5x/t5x/examples/decoder_only/models/xl.gin")
```

Since example `base_wmt_from_scratch.gin` relies on `base.gin` this issue might have been missed.

Contributor guide

Open the contributing guide

Research direction

Add the missing t5x/t5x/examples/decoder_only/models/__init__.py file. Start by running the gin.config.parse_config_file example with t5x/t5x/examples/decoder_only/models/xl.gin, and compare with the existing base_wmt_from_scratch.gin and base.gin relationship. Done means the configuration parses without the reported gin error.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
machine-learning
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.