NorwegianVeterinaryInstitute / DemultiplexRawSequenceData

A workflow automation script: demultiplex the library sequence, run quality checks, deliver to archiving and processing afterwards
GNU General Public License v3.0
1 stars 0 forks source link

bug: .md5 files and .sha512 files are not written correctly #71

Closed georgemarselis-nvi closed 1 week ago

georgemarselis-nvi commented 1 year ago

the format should be as the output of md5sum/sha512sum :

bnf-ish: HASH_2SPACES_filename

no _

to fix before 1.0

georgemarselis-nvi commented 1 year ago
                fh.write( f"{sha512sum}  {filetobehashed}\n" ) # the two spaces are mandatory to be re-verified after uploading via 'sha512sum -c FILE'

implemented a fix, re-running tests.

georgemarselis-nvi commented 1 year ago

image

looks good. closing ticket.

georgemarselis-nvi commented 1 year ago

something ate them changes.

re-implementing.

(manual for now) testing:

find /data/for_transfer -iname *.sha512 -exec /usr/bin/sha512sum -c /data/for_transfer/{ } \;

georgemarselis-nvi commented 1 year ago

image

works. fixed regression, but not in master branch, have to pull from the #68 branch.

georgemarselis-nvi commented 1 week ago

this was fix post haste