somehow fixed non-unified position IDs for the NAR-len
This commit is contained in:
parent
66407e5bdb
commit
26f74c5739
|
@ -438,8 +438,10 @@ class Base(nn.Module):
|
||||||
l_tokens = [n_resp_tokens] * (self.n_resp_levels + (1 if split_classifiers else 0))
|
l_tokens = [n_resp_tokens] * (self.n_resp_levels + (1 if split_classifiers else 0))
|
||||||
|
|
||||||
# there seems to be a problem with the NAR-only model with non-unified position IDs.............
|
# there seems to be a problem with the NAR-only model with non-unified position IDs.............
|
||||||
|
"""
|
||||||
if "len" in self.capabilities and not unified_position_ids:
|
if "len" in self.capabilities and not unified_position_ids:
|
||||||
raise Exception("ERROR: model instability for NAR-only model when not using unified position IDs.")
|
raise Exception("ERROR: model instability for NAR-only model when not using unified position IDs.")
|
||||||
|
"""
|
||||||
|
|
||||||
self.unified_position_ids = unified_position_ids
|
self.unified_position_ids = unified_position_ids
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue
Block a user