fix: back to share_vocab=True

rxn4chemistry · Apr 30, 2024 · 519a7c8 · 519a7c8
1 parent d9ec801
commit 519a7c8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/setup.cfg b/setup.cfg
@@ -30,7 +30,7 @@ install_requires =
     rxn-chem-utils>=1.1.4
     rxn-reaction-preprocessing>=2.0.2
     rxn-utils>=1.1.9
-    rxn-onmt-utils @ git+https://github.com/rxn4chemistry/rxn-onmt-utils.git@765d97e1371bc2ed00b4954defa8c4e14b5f86c5 #rxn-onmt-utils without rxn-opennmt-py depedency
+    rxn-onmt-utils @ git+https://github.com/rxn4chemistry/rxn-onmt-utils.git@f1a0b970411aac308a3cba36c942297933a4dd91 #rxn-onmt-utils without rxn-opennmt-py depedency
     OpenNMT-py>=3.5.1 # official onmt
 
 [options.packages.find]

diff --git a/src/rxn/onmt_models/scripts/rxn_onmt_preprocess.py b/src/rxn/onmt_models/scripts/rxn_onmt_preprocess.py
@@ -58,7 +58,7 @@ def get_build_vocab_config_file(
     valid_src: PathLike,
     valid_tgt: PathLike,
     save_data: Path,
-    share_vocab: bool = False,
+    share_vocab: bool = True,
     overwrite: bool = True,
     src_seq_length: int = 3000,
     tgt_seq_length: int = 3000,