IFIscripts icon indicating copy to clipboard operation
IFIscripts copied to clipboard

manifest_normalise.py script break

Open rcasey-ucc opened this issue 5 years ago • 2 comments

manifest_normalise.py breaks on macmini. works fine on es2 and yujings laptop. maybe it's an issue with macmini rather than anything else. if it is workign fine on es2 and laptops then we will be ok

terminal output: Last login: Thu Aug 6 11:50:47 on ttys002 macminis-mini:~ macmini$ manifest_normalise.py -h /Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 8: Normalises manifests so that they adhere to ifiscripts manifest. This includes removing lines with hashes. Removing lines with just whitespace. Creating a new sidecar called _modified_manifest.md5 The goal is to allow batchaccession to run on the new manifests. : command not found /Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 9: import: command not found /Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 10: import: command not found /Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 11: import: command not found /Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 12: import: command not found /Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 14: syntax error near unexpected token format='%(asctime)s - %(message)s',' /Users/macmini/ifigit/ifiscripts/manifest_normalise.py: line 14: logging.basicConfig(format='%(asctime)s - %(message)s', level=logging.DEBUG)' macminis-mini:~ macmini$

rcasey-ucc avatar Aug 06 '20 11:08 rcasey-ucc

Hey - so this was fixed locally on es2 but a problem happened when trying to push to github - so this fix never made its way up there. working on the fix now.

kieranjol avatar Aug 06 '20 13:08 kieranjol

I was able to push the fixes i made a few days ago so an update.py will sort this out now hopefully

kieranjol avatar Aug 06 '20 13:08 kieranjol