Closed parambharat closed 2 months ago
Check out this pull request on
See visual diffs & provide feedback on Jupyter Notebooks.
Powered by ReviewNB
@parambharat shouldn't we move the data
dir to an artifact or as weave dataset?
nit: how about we use the naming of the notebooks like - 00_name.ipynb
, 01_name1.ipynb
, etc?
The data that we are downloading contains only .md
file format?
🚨 Potential security issues detected. Learn more about Socket for GitHub ↗︎
To accept the risk, merge this PR and you will not be notified again.
Alert | Package | Note | Source | CI |
---|---|---|---|---|
Obfuscated code | pypi/orjson@3.10.7 |
| ⚠︎ |
Obfuscated files are intentionally packed to hide their behavior. This could be a sign of malware.
Packages should not obfuscate their code. Consider not using packages with obfuscated code
Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev.
If you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency.
To ignore an alert, reply with a comment starting with @SocketSecurity ignore
followed by a space separated list of ecosystem/package-name@version
specifiers. e.g. @SocketSecurity ignore npm/foo@1.0.0
or ignore all packages with @SocketSecurity ignore-all
@SocketSecurity ignore pypi/orjson@3.10.7
New dependencies detected. Learn more about Socket for GitHub ↗︎
This PR contains the course work and related code for the RAG-IRL course.