Skip to content

slurm script time interval, Ume config, ModernBERT max length#59

Merged
ncfrey merged 4 commits intomainfrom
ume-config-update
Apr 1, 2025
Merged

slurm script time interval, Ume config, ModernBERT max length#59
ncfrey merged 4 commits intomainfrom
ume-config-update

Conversation

@karinazad
Copy link
Collaborator

No description provided.


datamodule = hydra.utils.instantiate(cfg.data)
datamodule.prepare_data()
datamodule.setup(stage="fit")
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

avoid preparing data twice since trainer calls this too

@karinazad karinazad changed the title slurm script time interval, Ume config slurm script time interval, Ume config, ModernBERT max length Apr 1, 2025
@ncfrey ncfrey merged commit 44e030e into main Apr 1, 2025
5 checks passed
@ncfrey ncfrey deleted the ume-config-update branch April 1, 2025 18:07
taylormjs pushed a commit that referenced this pull request Apr 2, 2025
* config

* max length

* max length

* beignet
taylormjs added a commit that referenced this pull request Apr 2, 2025
* add iid splits, heldout set

* ruff

* update docstring

* Ume evaluation callbacks - switch to Ume tokenizers, expose `evaluate` (#57)

* fix callbacks

* name

* fix

* callbacks

* docstrings

* fix

* notebook

* temp datamodule

* remove default caching in homefs

* update tests, fix ruff

* slurm script time interval, Ume config, ModernBERT max length (#59)

* config

* max length

* max length

* beignet

* configs (#55)

Co-authored-by: freyn6 <freyn6@gene.com>

* make iid test set, fix heldout to combine all species, update all supported splits

* fix iterable split getting

---------

Co-authored-by: Taylor Joren <joren.taylor@gene.com>
Co-authored-by: Karina Zadorozhny <karina.zadorozhny@gmail.com>
Co-authored-by: Nathan Frey <ncfrey@users.noreply.github.com>
Co-authored-by: freyn6 <freyn6@gene.com>
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