Skip to content

Add --max-model-len & add Laya multi-lingual and typed-decisions - #13

Merged
alvarobartt merged 3 commits into
mainfrom
add-missing-laya-variants
Sep 25, 2026
Merged

alvarobartt merged 3 commits into
mainfrom
add-missing-laya-variants

Conversation

@alvarobartt

Copy link
Copy Markdown
Owner

Description

This PR adds the --max-model-len flag which defaults to the max_len value set in in the rl_agent_config.json file for Laya, whereas usually the encoder context length is defined in max_position_embeddings in the config.json (in this case placed under encoder/config.json).

Additionally, this PR adds the missing multi-lingual and typed-decisions Laya variants other than the English-only one; which is linked with the --max-model-len addition, since for the multi-lingual version, despite being trained with sequences up to 1024 setting --max-model-len 8192 is recommended to leverage a larger context length, as the encoder supports sequences up to 8192 (as set in its config.json under encoder/config.json).

Finally, it also includes some slight changes in the GitHub Actions to make sure that the cache is reused efficiently and that's only triggered when applicable.

@alvarobartt
alvarobartt force-pushed the add-missing-laya-variants branch from 0485758 to 9296aa1 Compare September 24, 2026 20:28
@alvarobartt
alvarobartt added this pull request to stack #15 September 25, 2026 07:48
@alvarobartt
alvarobartt merged commit a5e7e84 into main Sep 25, 2026
16 checks passed
@alvarobartt
alvarobartt deleted the add-missing-laya-variants branch September 25, 2026 13:14

This branch was successfully deployed

1 active deployment
pr-build — 9296aa1a Deployed Sep 24, 2026 by alvarobartt via Approve PR build #4
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.

1 participant