Skip to content

Commit

Permalink
Merge branch 'main' into hey-my-first-pr-to-olmo
Browse files Browse the repository at this point in the history
  • Loading branch information
2015aroras authored Oct 2, 2024
2 parents bdf17f0 + aa1863e commit 2256ff5
Show file tree
Hide file tree
Showing 3 changed files with 57 additions and 52 deletions.
3 changes: 2 additions & 1 deletion hf_olmo/configuration_olmo.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
from transformers.utils import logging

from olmo.config import ModelConfig
from olmo.exceptions import OLMoConfigurationError

logger = logging.get_logger(__name__)

Expand Down Expand Up @@ -55,7 +56,7 @@ def effective_n_kv_heads(self) -> int:
else:
raise OLMoConfigurationError(
"You can't set `multi_query_attention` and `n_kv_heads` at the same time."
)
)


# Register the config class so that it is available for transformer pipelines, auto-loading etc.
Expand Down
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ authors = [
requires-python = ">=3.8"
license = { file = "LICENSE" }
dependencies = [
"numpy",
"numpy<2",
"torch>=2.1,<2.5",
"ai2-olmo-core==0.1.0",
"omegaconf",
Expand Down
104 changes: 54 additions & 50 deletions scripts/ladder.ipynb

Large diffs are not rendered by default.

0 comments on commit 2256ff5

Please sign in to comment.