• Type: New Feature
• Resolution: Unresolved
• Priority: Normal
• Fix Version/s: None
• Affects Version/s: None
• Component/s: Tags & Metadata
• Labels:

  • good-first-bug
  • plugins

[PICARD-2945] Provide broadcast date variable

Collapse comment: Philipp Wolfer added a comment - 2024-07-30 07:56

Philipp Wolfer added a comment - 2024-07-30 07:56

Oh nice, thanks for sharing.

Generally this makes sense and if we integrate this into Picard we can directly include the label rels in the original request. Currently I don't think this feature has much priority as we are working on larger refactorings for Picard 3, but the feature makes sense. Hence I also suggested a plugin, because this gives an immediate solution. But if someone wants to work on a patch for Picard I'd be happy to have this reviewed and merged.

My suggestion would be to store the broadcast date in metadata as something like "~broadcast_date" (in scripting it would then be available as "%broadcast_date%", as I don't think there is a commonly used tag field for this. But making it available to scripting gives the flexibility to use the field as the user prefers.

Expand comment: Philipp Wolfer added a comment - 2024-07-30 07:56

Philipp Wolfer added a comment - 2024-07-30 07:56 Oh nice, thanks for sharing. Generally this makes sense and if we integrate this into Picard we can directly include the label rels in the original request. Currently I don't think this feature has much priority as we are working on larger refactorings for Picard 3, but the feature makes sense. Hence I also suggested a plugin, because this gives an immediate solution. But if someone wants to work on a patch for Picard I'd be happy to have this reviewed and merged. My suggestion would be to store the broadcast date in metadata as something like "~broadcast_date" (in scripting it would then be available as "%broadcast_date%", as I don't think there is a commonly used tag field for this. But making it available to scripting gives the flexibility to use the field as the user prefers.

Collapse comment: diskotechjam added a comment - 2024-07-30 02:19, Edited by diskotechjam - 2024-07-30 02:19

diskotechjam added a comment - 2024-07-30 02:19 - edited

I've actually already written one (first plugin though, so I'm sure it could be done better): https://github.com/tfranken90/picard-broadcastdates

Current guidelines distinguish between broadcast and release date, so this seems to be the only way to get the value without parsing the recording title (which isn't currently an option for music videos). 

Expand comment: diskotechjam added a comment - 2024-07-30 02:19, Edited by diskotechjam - 2024-07-30 02:19

diskotechjam added a comment - 2024-07-30 02:19 - edited I've actually already written one (first plugin though, so I'm sure it could be done better): https://github.com/tfranken90/picard-broadcastdates Current guidelines distinguish between broadcast and release date, so this seems to be the only way to get the value without parsing the recording title (which isn't currently an option for music videos). 

Collapse comment: Philipp Wolfer added a comment - 2024-07-28 22:59

Philipp Wolfer added a comment - 2024-07-28 22:59

Thanks for the suggestion. This is something that could also be solved with a plugin, if someone wants to give this a go.

Expand comment: Philipp Wolfer added a comment - 2024-07-28 22:59

Philipp Wolfer added a comment - 2024-07-28 22:59 Thanks for the suggestion. This is something that could also be solved with a plugin, if someone wants to give this a go.