Skip to content
This repository was archived by the owner on Dec 16, 2022. It is now read-only.

Commit c37ff2c

Browse files
authored
update config files to jsonnet format (#1479)
* update config files to jsonnet format * rename json -> jsonnet
1 parent f3fce4c commit c37ff2c

File tree

6 files changed

+17
-9
lines changed

6 files changed

+17
-9
lines changed

allennlp/tests/common/params_test.py

+9-1
Original file line numberDiff line numberDiff line change
@@ -192,12 +192,20 @@ def test_env_var_substitution(self):
192192
def test_known_configs(self):
193193
configs = os.listdir(self.PROJECT_ROOT / "training_config")
194194

195+
# Our configs use environment variable substitution, and the _jsonnet parser
196+
# will fail if we don't pass it correct environment variables.
195197
forced_variables = [
196198
# constituency parser
197199
'PTB_TRAIN_PATH', 'PTB_DEV_PATH', 'PTB_TEST_PATH',
198200

199201
# srl_elmo_5.5B
200-
'SRL_TRAIN_DATA_PATH', 'SRL_VALIDATION_DATA_PATH'
202+
'SRL_TRAIN_DATA_PATH', 'SRL_VALIDATION_DATA_PATH',
203+
204+
# coref
205+
'COREF_TRAIN_DATA_PATH', 'COREF_DEV_DATA_PATH', 'COREF_TEST_DATA_PATH',
206+
207+
# ner
208+
'NER_TRAIN_DATA_PATH', 'NER_TEST_A_PATH', 'NER_TEST_B_PATH'
201209
]
202210

203211
for var in forced_variables:

training_config/coref.json renamed to training_config/coref.jsonnet

+3-3
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,9 @@
1212
},
1313
"max_span_width": 10
1414
},
15-
"train_data_path":"${COREF_TRAIN_DATA_PATH}",
16-
"validation_data_path": "${COREF_DEV_DATA_PATH}",
17-
"test_data_path": "${COREF_TEST_DATA_PATH}",
15+
"train_data_path": std.extVar("COREF_TRAIN_DATA_PATH"),
16+
"validation_data_path": std.extVar("COREF_DEV_DATA_PATH"),
17+
"test_data_path": std.extVar("COREF_TEST_DATA_PATH"),
1818
"model": {
1919
"type": "coref",
2020
"text_field_embedder": {

training_config/ner.json renamed to training_config/ner.jsonnet

+3-3
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,9 @@
1616
}
1717
}
1818
},
19-
"train_data_path": "${NER_TRAIN_DATA_PATH}",
20-
"validation_data_path": "${NER_TEST_A_PATH}",
21-
"test_data_path": "${NER_TEST_B_PATH}",
19+
"train_data_path": std.extVar("NER_TRAIN_DATA_PATH"),
20+
"validation_data_path": std.extVar("NER_TEST_A_PATH"),
21+
"test_data_path": std.extVar("NER_TEST_B_PATH"),
2222
"evaluate_on_test": true,
2323
"model": {
2424
"type": "crf_tagger",

training_config/semantic_role_labeler.json renamed to training_config/semantic_role_labeler.jsonnet

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"dataset_reader":{"type":"srl"},
3-
"train_data_path": "${SRL_TRAIN_DATA_PATH}",
4-
"validation_data_path": "${SRL_VALIDATION_DATA_PATH}",
3+
"train_data_path": std.extVar("SRL_TRAIN_DATA_PATH"),
4+
"validation_data_path": std.extVar("SRL_VALIDATION_DATA_PATH"),
55
"model": {
66
"type": "srl",
77
"text_field_embedder": {

0 commit comments

Comments
 (0)