Merge pull request #80 from graphcore/Fix

Change addons requirements to https, disable wandb by default
graphcore · Jan 13, 2023 · 5d96fd5 · 5d96fd5
2 parents 03e04e8 + 6071c07
commit 5d96fd5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/nlp/gpt_j/popxl/requirements.txt b/nlp/gpt_j/popxl/requirements.txt
@@ -15,6 +15,6 @@ sklearn==0.0
 pytest==6.2.5
 pytest-pythonpath==0.7.4
 
-git+ssh://git@github.com/graphcore/[email protected]_a
+git+https://github.com/graphcore/[email protected]_a
 
 protobuf==3.20.*; python_version > '3.6'
diff --git a/nlp/gpt_j/popxl/utils/setup.py b/nlp/gpt_j/popxl/utils/setup.py
@@ -124,7 +124,7 @@ def gptj_fine_tuning_setup(
 ) -> Tuple[GPTJConfig, argparse.Namespace, Optional[GPTJForCausalLM]]:
  """GPT-J setup for fine tunning scripts"""
  config, args, pretrained = gptj_config_setup(config_file, presets_key, default_config,
- wandb_setup=True, hf_model_setup=True)
+ wandb_setup=False, hf_model_setup=True)
 
  return config, args, pretrained