mrmiguez / citrus Goto Github PK

View Code? Open in Web Editor NEW

3.0 3.0 1.0 735 KB

Collective Information Transformation and Reconciliation Utility Service

License: MIT License

Python 100.00%

citrus's People

Contributors

Stargazers

Watchers

Forkers

paclough

citrus's Issues

Fix for memorization

Learning to use generators might be more efficient: https://stackoverflow.com/questions/231767/what-does-the-yield-keyword-do

Strip null values?

Might not be necessary

Flexible profile selection

Right now everything is hard-coded to use 'SSDN' config sections.

exceptions tests

Finish generic commands for subcommands

-n --new, -l --list, -i --interactive

Identifiers

Don't pass up any local identifier to DPLA
Select either PURL or OAI ID for SR.identifier

Not all future participants will have PURLs available

Include additional QDC date types

Currenly FlaLD_QDC is only checking dcterms:created for sourceResource.date.

There's many more that need to be included: http://dublincore.org/documents/dcmi-terms/#section-2

FIU thumbnails

Change tn_service for FIU to look in thumbnail elem, rather than construct SR.preview from Sobek identifier.

Rewrite README

Describe citrus module classes. Describe use of new CLI.

Reimplement logging

and do it better

CLI main tests

Tests for harvest and transform called by __main__, other citrus.cli functions

dPanther PURLs

Possibly affecting SR.identifier in addition to aggregation.preview

From error log:

WARNING:root:aggregation.preview: local variable 'PURL_match' referenced before assignment - oai:lib.fsu.edu.fiu_ps:oai:uofm.library.fiu_ps:oai:dpsobek:FISI001546_00001
WARNING:root:aggregation.preview: local variable 'PURL_match' referenced before assignment - oai:lib.fsu.edu.fiu_ps:oai:uofm.library.fiu_ps:oai:dpsobek:FISI001556_00001
WARNING:root:aggregation.preview: local variable 'PURL_match' referenced before assignment - oai:lib.fsu.edu.fiu_ps:oai:uofm.library.fiu_ps:oai:dpsobek:FISI001566_00001