Skip to content

Commit 59961d2

Browse files
committed
update deprecated log metadata command
1 parent c4e6224 commit 59961d2

File tree

2 files changed

+15
-9
lines changed

2 files changed

+15
-9
lines changed

llm-finetuning/steps/deployment.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ def deploy_model_to_hf_hub(hf_endpoint_cfg: Optional[Dict] = None) -> None:
4040
hf_token = secret.secret_values["token"]
4141

4242
commit_info = (
43-
get_step_context().model.run_metadata["merged_model_commit_info"].value
43+
get_step_context().model.run_metadata["merged_model_commit_info"]
4444
)
4545

4646
model_namespace, repository, revision = parse_huggingface_url(commit_info)

llm-finetuning/steps/trainer.py

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
)
3636
from transformers.models.gpt2.tokenization_gpt2_fast import GPT2TokenizerFast
3737
from typing_extensions import Annotated
38-
from zenml import ArtifactConfig, log_model_metadata, save_artifact, step
38+
from zenml import ArtifactConfig, log_metadata, save_artifact, step
3939
from zenml.client import Client
4040
from zenml.enums import ArtifactType
4141

@@ -524,8 +524,9 @@ def run_training(args: Configuration, train_data, val_data, hf_token):
524524
try:
525525
if args.push_to_hub:
526526
commit_info = trainer.push_to_hub()
527-
log_model_metadata(
528-
metadata={"trainer_commit_info": str(commit_info)}
527+
log_metadata(
528+
metadata={"trainer_commit_info": str(commit_info)},
529+
infer_model=True
529530
)
530531
else:
531532
trainer.save_model(args.output_dir)
@@ -535,8 +536,9 @@ def run_training(args: Configuration, train_data, val_data, hf_token):
535536
commit_info = trainer.model.push_to_hub(
536537
repo_id=args.output_peft_repo_id, token=hf_token
537538
)
538-
log_model_metadata(
539-
metadata={"model_commit_info": str(commit_info)}
539+
log_metadata(
540+
metadata={"model_commit_info": str(commit_info)},
541+
infer_model=True
540542
)
541543
except Exception as e:
542544
print("Exception while pushing or saving")
@@ -580,11 +582,15 @@ def merge_and_push(
580582

581583
model_id_merged = f"{peft_model_id}-merged"
582584
commit_info = tokenizer.push_to_hub(model_id_merged, token=hf_token)
583-
log_model_metadata(
584-
metadata={"merged_tokenizer_commit_info": str(commit_info)}
585+
log_metadata(
586+
metadata={"merged_tokenizer_commit_info": str(commit_info)},
587+
infer_model=True
585588
)
586589
commit_info = final_model.push_to_hub(model_id_merged, token=hf_token)
587-
log_model_metadata(metadata={"merged_model_commit_info": str(commit_info)})
590+
log_metadata(
591+
metadata={"merged_model_commit_info": str(commit_info)},
592+
infer_model=True
593+
)
588594

589595

590596
@step(output_materializers={"trainer_obj": HFTrainerMaterializer})

0 commit comments

Comments
 (0)