Repository: facebookresearch/Mephisto

Issue:
Running static_test_script.py: Usage of deprecated keyword in package header
Seems like this is a compatibility issue with hydra 1.1.0? I've changed static_test_script.py like so:
`@hydra.main(config_path=".", config_name="scriptconfig")`
and deleted the first line config/example.yaml as instructed by hydra.cc, after running the script I'm still getting the following error:
`/home/ninacdu/.local/lib/python3.8/site-packages/hydra/core/default_element.py:122: UserWarning: In 'mephisto/provider/mock': Usage of deprecated keyword in package header '# @package _group_'.
See https://hydra.cc/docs/next/upgrades/1.0_to_1.1/changes_to_package_header for more information
  warnings.warn(
/home/ninacdu/.local/lib/python3.8/site-packages/hydra/core/default_element.py:122: UserWarning: In 'mephisto/architect/local': Usage of deprecated keyword in package header '# @package _group_'.
See https://hydra.cc/docs/next/upgrades/1.0_to_1.1/changes_to_package_header for more information
  warnings.warn(
/home/ninacdu/.local/lib/python3.8/site-packages/hydra/core/default_element.py:122: UserWarning: In 'mephisto/blueprint/static_task': Usage of deprecated keyword in package header '# @package _group_'.
See https://hydra.cc/docs/next/upgrades/1.0_to_1.1/changes_to_package_header for more information
  warnings.warn(
In 'scriptconfig': Validation error while composing config:
Merge error: BlueprintArgs is not a subclass of StaticHTMLBlueprintArgs. value: {'_blueprint_type': '???', 'onboarding_qualification': '???', 'block_qualification': '???'}
    full_key: 
    object_type=dict

Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.
`
Is there something I'm doing wrong here? 


Discussion:
I hadn't realized 1.1.0 was out yet - excited to upgrade, but for now Mephisto isn't compatible yet. 
Thanks for the reply! I downgraded hydra to 1.0.6, and after running the script the error I got is as follow:
`[2021-06-28 18:42:20,597][mephisto.operations.operator][ERROR] - Ran into error while launching run: 
Traceback (most recent call last):
  File "/media/ninacdu/Workspace/convqa/Mephisto/mephisto/operations/operator.py", line 424, in validate_and_run_config
    return self.validate_and_run_config_or_die(
  File "/media/ninacdu/Workspace/convqa/Mephisto/mephisto/operations/operator.py", line 166, in validate_and_run_config_or_die
    BlueprintClass.assert_task_args(run_config, shared_state)
  File "/media/ninacdu/Workspace/convqa/Mephisto/mephisto/abstractions/blueprints/static_html_task/static_html_blueprint.py", line 111, in assert_task_args
    if isinstance(shared_state.static_task_data, types.GeneratorType):
AttributeError: 'SharedTaskState' object has no attribute 'static_task_data'
[2021-06-28 18:42:20,597][mephisto.operations.operator][INFO] - operator shutting down
`
What should I do here?
I'm not able to reproduce this locally unfortunately, however I'm on `hydra-core==1.0.0` - I'll be working to upgrade to `1.1.0` tomorrow.
Hi @ninadu07 - after cleaning up my branch I was able to reproduce and resolve this issue, am going to push a fix for this task.

Resolving patch:
diff --git a/examples/simple_static_task/static_run_with_onboarding.py b/examples/simple_static_task/static_run_with_onboarding.py
--- a/examples/simple_static_task/static_run_with_onboarding.py
+++ b/examples/simple_static_task/static_run_with_onboarding.py
@@ -43,7 +43,7 @@ class TestScriptConfig(RunScriptConfig):
 register_script_config(name="scriptconfig", module=TestScriptConfig)
 
 
-@hydra.main(config_name="scriptconfig")
+@hydra.main(config_path="hydra_configs", config_name="scriptconfig")
 def main(cfg: DictConfig) -> None:
     correct_config_answer = cfg.correct_answer
 
diff --git a/examples/static_react_task/run_task.py b/examples/static_react_task/run_task.py
--- a/examples/static_react_task/run_task.py
+++ b/examples/static_react_task/run_task.py
@@ -71,7 +71,7 @@ def build_task(task_dir):
     os.chdir(return_dir)
 
 
-@hydra.main(config_name="scriptconfig")
+@hydra.main(config_path="hydra_configs", config_name="scriptconfig")
 def main(cfg: DictConfig) -> None:
     task_dir = cfg.task_dir
 
diff --git a/mephisto/operations/hydra_config.py b/mephisto/operations/hydra_config.py
--- a/mephisto/operations/hydra_config.py
+++ b/mephisto/operations/hydra_config.py
@@ -23,9 +23,9 @@ class DatabaseArgs:
 
 @dataclass
 class MephistoConfig:
-    blueprint: BlueprintArgs = BlueprintArgs()
-    provider: ProviderArgs = ProviderArgs()
-    architect: ArchitectArgs = ArchitectArgs()
+    blueprint: BlueprintArgs = MISSING
+    provider: ProviderArgs = MISSING
+    architect: ArchitectArgs = MISSING
     task: TaskConfigArgs = TaskConfigArgs()
     database: DatabaseArgs = DatabaseArgs()
     log_level: str = "info"
@@ -38,7 +38,9 @@ class RunScriptConfig:
 
 def register_abstraction_config(name: str, node: Any, abstraction_type: str):
     config.store(
-        name=name, node=node, group=f"mephisto/{abstraction_type}", package="_group_"
+        name=name,
+        node=node,
+        group=f"mephisto/{abstraction_type}",
     )
 
 
@@ -50,7 +52,6 @@ def initialize_named_configs():
         name="base_mephisto_config",
         node=MephistoConfig,
         group="mephisto",
-        package="_group_",
     )
 
 


Test patch:
diff --git a/examples/parlai_chat_task_demo/parlai_test_script.py b/examples/parlai_chat_task_demo/parlai_test_script.py
--- a/examples/parlai_chat_task_demo/parlai_test_script.py
+++ b/examples/parlai_chat_task_demo/parlai_test_script.py
@@ -51,7 +51,7 @@ class TestScriptConfig(RunScriptConfig):
 register_script_config(name="scriptconfig", module=TestScriptConfig)
 
 
-@hydra.main(config_name="scriptconfig")
+@hydra.main(config_path="hydra_configs", config_name="scriptconfig")
 def main(cfg: DictConfig) -> None:
     db, cfg = load_db_and_process_config(cfg)
 
diff --git a/examples/simple_static_task/static_test_script.py b/examples/simple_static_task/static_test_script.py
--- a/examples/simple_static_task/static_test_script.py
+++ b/examples/simple_static_task/static_test_script.py
@@ -39,7 +39,7 @@ class TestScriptConfig(RunScriptConfig):
 register_script_config(name="scriptconfig", module=TestScriptConfig)
 
 
-@hydra.main(config_name="scriptconfig")
+@hydra.main(config_path="hydra_configs", config_name="scriptconfig")
 def main(cfg: DictConfig) -> None:
     db, cfg = load_db_and_process_config(cfg)
     operator = Operator(db)