Spaces:
Sleeping
Sleeping
Fix a bug with conflicted speakers in the same rename
Browse files
ui.py
CHANGED
|
@@ -138,7 +138,7 @@ def render_rename_sidebar(currFile, speakerNames, display_speaker_tokens, token_
|
|
| 138 |
|
| 139 |
st.sidebar.text(" ")
|
| 140 |
|
| 141 |
-
# Build claimed set (display labels) from the data model
|
| 142 |
all_claimed = {
|
| 143 |
raw_to_display.get(token, token)
|
| 144 |
for entry in st.session_state.globalRenames
|
|
@@ -172,6 +172,14 @@ def render_rename_sidebar(currFile, speakerNames, display_speaker_tokens, token_
|
|
| 172 |
args=(idx, token_display_map),
|
| 173 |
label_visibility="collapsed",
|
| 174 |
)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 175 |
|
| 176 |
st.sidebar.text_input(
|
| 177 |
"Add rename", placeholder="e.g. John",
|
|
|
|
| 138 |
|
| 139 |
st.sidebar.text(" ")
|
| 140 |
|
| 141 |
+
# Build claimed set (display labels) from the data model — one speaker per entry
|
| 142 |
all_claimed = {
|
| 143 |
raw_to_display.get(token, token)
|
| 144 |
for entry in st.session_state.globalRenames
|
|
|
|
| 172 |
args=(idx, token_display_map),
|
| 173 |
label_visibility="collapsed",
|
| 174 |
)
|
| 175 |
+
# Show error if two speakers from the same file are in this entry
|
| 176 |
+
files_in_entry = [token.split(": ", 1)[0] for token in entry["speakers"]]
|
| 177 |
+
for fname in set(files_in_entry):
|
| 178 |
+
if files_in_entry.count(fname) > 1:
|
| 179 |
+
st.sidebar.error(
|
| 180 |
+
f"Conflicted name for two speakers in one same file. Please remove one speaker."
|
| 181 |
+
)
|
| 182 |
+
break
|
| 183 |
|
| 184 |
st.sidebar.text_input(
|
| 185 |
"Add rename", placeholder="e.g. John",
|