Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TUNOCENT recording list bugs / improvements #211

Open
5 of 7 tasks
dasch124 opened this issue Jan 15, 2025 · 4 comments · Fixed by #217
Open
5 of 7 tasks

TUNOCENT recording list bugs / improvements #211

dasch124 opened this issue Jan 15, 2025 · 4 comments · Fixed by #217
Assignees
Labels

Comments

@dasch124
Copy link
Member

dasch124 commented Jan 15, 2025

Bugs / open Issues - please fix

  • "Interviewer" always seems to be "VB" ? (please use recordingStmt.recording.respStmt[resp='recording'].persName (simpleTEIMetadata.resp seems to be populated by the first respStmt (which is "resp=principal" (i.e. the PI), and not the responsible researcher)
  • rename "Settlement" to "Location"
  • "Region" column is duplicated

Less important improvements after the initial release:

Nice-to-haves

sometimes/maybe

  • Place, Region, Speaker could be comboboxes / autocompletes
  • indicate audio availability from TEI.teiHeader.fileDesc.sourceDesc.recordingStmt.tei:recording.media[@type='master']/@decls (this is a pointer to an availability element with status="restricted" or "free") - ideally visually similar to replace audio file name with symbol #214
  • new column "TEI" with the title of processed TEI document and link to the document's detail view; with "TEI yes/no"
@MauPalantir
Copy link
Contributor

columns were committed to the map - data table filters connection branch as I was working with that file, sry.

@dasch124
Copy link
Member Author

dasch124 commented Jan 29, 2025

NB It seems that in the recordings list only titles of lingfeatures are displayed as links to the data point. If that's an easy fix, could you please revisit that, @MauPalantir ?

@MauPalantir
Copy link
Contributor

Sure!

@MauPalantir
Copy link
Contributor

MauPalantir commented Feb 5, 2025

@dasch124 Fixed the ST links missing in b28fe76

Will have to add the corpus texts which might be a bit different logic, but will do so as well.

for the last 2 checkboxes, the TEI column would only mean whether we have a transcription?
And would it point to the transcription link or the TEI source? It the former, that sounds a bit redundant because we would already have the transcription link in the title.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants