-
Notifications
You must be signed in to change notification settings - Fork 2
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
00 sort bi extracts #837
00 sort bi extracts #837
Conversation
…nd/source-linkage-files into 00_Sort_BI_Extracts
…nd/source-linkage-files into 00_Sort_BI_Extracts
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[1] "source-episode-file-2223.parquet" "source-individual-file-2223.parquet"
Error: [ENOENT] Failed to copy '/conf/sourcedev/Source_Linkage_File_Updates/2223/source-episode-file-2223.parquet' to '/conf/hscdiip/copy_test/source-episode-file-2223.parquet': no such file or directory
When trying to test one year, i get this error even though i created a copy_test
folder in hscdiip. Could this be a permissions issue?
This comment has been minimized.
This comment has been minimized.
@check-spelling-bot Report🔴 Please reviewSee the 📂 files view, the 📜action log, or 📝 job summary for details. Unrecognized words (4)beddays To accept these unrecognized words as correct, you could run the following commands... in a clone of the [email protected]:Public-Health-Scotland/source-linkage-files.git repository curl -s -S -L 'https://raw.githubusercontent.com/check-spelling/check-spelling/main/apply.pl' |
perl - 'https://github.com/Public-Health-Scotland/source-linkage-files/actions/runs/6393346910/attempts/1' OR To have the bot accept them for you, reply quoting the following line: Available 📚 dictionaries could cover words (expected and unrecognized) not in the 📘 dictionaryThis includes both expected items (232) from .github/actions/spelling/expect.txt and unrecognized words (4)
Consider adding them (in with:
extra_dictionaries:
cspell:k8s/dict/k8s.txt
cspell:npm/dict/npm.txt To stop checking additional dictionaries, add (in check_extra_dictionaries: '' Errors (2)See the 📂 files view, the 📜action log, or 📝 job summary for details.
See ❌ Event descriptions for more information. If the flagged items are 🤯 false positivesIf items relate to a ...
|
It is a permission issue. We should use "dir.create(file_path)" to create a folder. |
Should be good to merge this time |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Look good! Will test it tomorrow and see if I get any permission issues
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All seems good. We can test properly when we get the next boxi extracts
Use fs::file_copy to replace file.copy to improve the speed