Skip to content

Commit ac5bfe5

Browse files
Merge pull request #99 from patrickfleith/bug/small-bug-fixes
Bug/small bug fixes
2 parents ea017ba + 08520bd commit ac5bfe5

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

datafast/datasets.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,7 @@ def push_to_hub(
235235
self,
236236
repo_id: str,
237237
token: str | None = None,
238-
private: bool = False,
238+
private: bool = True,
239239
commit_message: str | None = None,
240240
train_size: float | None = None,
241241
seed: int | None = None,

datafast/utils.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,8 @@ def _get_mcq_specific_factors(config: MCQDatasetConfig) -> dict[str, int]:
120120

121121
def _get_mcq_num_expected_rows(config: MCQDatasetConfig, llms: list[LLMProvider], source_data_num_rows: int) -> int:
122122
# factors = _get_mcq_specific_factors(config) # Not specific factors
123-
source_data_num_rows = min(source_data_num_rows, config.sample_count)
123+
if config.sample_count is not None:
124+
source_data_num_rows = min(source_data_num_rows, config.sample_count)
124125
num_llms = len(llms)
125126
if config.prompts is None:
126127
num_expanded_prompts = 1

0 commit comments

Comments
 (0)