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

sipcreator - add accession option #325

Merged
merged 2 commits into from
Mar 8, 2019
Merged

sipcreator - add accession option #325

merged 2 commits into from
Mar 8, 2019

Conversation

kieranjol
Copy link
Owner

@kieranjol kieranjol commented Feb 11, 2019

This combines the OE and accession process in the cases where:

  • QC has already been carried out
  • The files are definitely going to be accessioned.
  • an accession number and a OE number are pre-block booked
  • the filmographic already exists

This also generates a helper OE record that fills in OE number, title, accession number, acquisition info, and it automates the 'Representation of X|Reproduction of X'.

It should significantly reduce the admin required for object entry-ing/accessioning objects that we know will definitely be accessioned.

Some extra checks need to be added - like an initial check to see that the reference number actually exists in the filmographic.

Feedback very welcome @raecasey @ecodonohoe .

@kieranjol
Copy link
Owner Author

It also makes sense to check that the -csv option is used alongside -accession as both are necessary. I think i had problems in the past trying to force a second option if one option is used in argparse, so perhaps just prompt the user for the path instead.

@kieranjol
Copy link
Owner Author

It's easy to use mergepbcore to make the database import CSV, but generating all the accession register records can be a bit trickier as they are now all individual files.
I have a workaround where i use bash tools like cat and sort to make the helper file which is all fairly painless, but something else might need to be added in python.

@kieranjol kieranjol merged commit fa9016d into master Mar 8, 2019
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

Successfully merging this pull request may close these issues.

None yet

1 participant