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

makepbcore.py -include value =n/a for multiple fields #261

Closed
raecasey opened this issue May 4, 2018 · 4 comments
Closed

makepbcore.py -include value =n/a for multiple fields #261

raecasey opened this issue May 4, 2018 · 4 comments

Comments

@raecasey
Copy link
Contributor

raecasey commented May 4, 2018

here are the fields where this needs to be included:
instantiationDimensi
instantiationLocatio
instantTimeStart
instantColors
instantLanguage
instantiationChanCon

@kieranjol
Copy link
Owner

All but instantiationDimensi are implemented locally.
I think for this one, we said that we wouldn't use it at all, as the information is covered elsewhere? Also it's a combination of height and scan type (1080p, 720p). I think it was a mistake that we left it in as included in the XDCAM format spreadsheet. Same for DCP.. I think that this could be removed from the database and not be included at all..

Also for instantiationLocatio - I can't remember how we would use this. The 'habitat' field in our database maps to this PBCore field, so having a seperate instantiationLocatio field might be a duplication of effort, unless we want to use this for barcodes or something? I'll check the minutes of our last meeting to see what was decided..

@raecasey
Copy link
Contributor Author

yes re instantiationlocation. - it is the habitat field. we put it in as a yes, because we decided it was the same as the habitat field. i shouldn't have put it in the list above, sorry
R

@raecasey
Copy link
Contributor Author

had a look over the pbcore template again there, - i think you are right. i think we said yes to dcp and xdcam but then re-thought it when it came to concat and converted, - but either neglected to revisit the subject for dcp and xdcam or just updating the field.
anyway, - it can be taken off the list and i'll adjust the pbcore template
R

@kieranjol
Copy link
Owner

Savage. I've removed instantiationLocation and instantiationDimension just there, and all of the other fields produce 'n/a' for now. I'm gonna close this but re-open if necessary.

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

No branches or pull requests

2 participants