Skip to content

Commit

Permalink
rename infer_test_set for greater clarity
Browse files Browse the repository at this point in the history
  • Loading branch information
benjaminye committed Apr 8, 2024
1 parent 557719f commit bc83065
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion llmtune/cli/toolkit.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ def run_one_experiment(config: Config, config_path: str) -> None:
results_file_path = join(dir_helper.save_paths.results, "results.csv")
if not exists(results_path) or exists(results_file_path):
inference_runner = LoRAInference(test, test_column, config, dir_helper)
inference_runner.infer_all()
inference_runner.infer_test_set()
RichUI.after_inference(results_path)
else:
RichUI.inference_found(results_path)
Expand Down
2 changes: 1 addition & 1 deletion llmtune/inference/generics.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ def infer_one(self, prompt: str):
pass

@abstractmethod
def infer_all(self):
def infer_test_set(self):
pass
2 changes: 1 addition & 1 deletion llmtune/inference/lora.py
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ def _get_merged_model(self, weights_path: str):

return model, tokenizer

def infer_all(self):
def infer_test_set(self):
results = []
prompts = self.test_dataset["formatted_prompt"]
labels = self.test_dataset[self.label_column]
Expand Down

0 comments on commit bc83065

Please sign in to comment.