Pin scikit-learn version to fix vulnerability in legacy/requirements.txt
#4
Conversation
The following vulnerabilities are fixed by pinning transitive dependencies: - https://snyk.io/vuln/SNYK-PYTHON-SCIKITLEARN-7217830
|
Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information |
|
Unable to locate .performanceTestingBot config file |
|
Thanks @2lambda123 for opening this PR! For COLLABORATOR only :
|
![gitginie[bot]](https://avatars.githubusercontent.com/in/712912?s=60&v=4){kind=small}
There was a problem hiding this comment.
@2lambda123
Thank you for your contribution to this repository! We appreciate your effort in opening pull request.
Happy coding!
|
Processing PR updates... |
Their most recently public accepted PR is: 2lambda123/optics-dev-Monocly#1 |
|
Important Review skippedIgnore keyword(s) in the title. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
labels-and-badges
bot
added
NO JIRA
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
We have skipped reviewing this pull request. Here's why:
- It seems to have been created by a bot ('[Snyk]' found in title). We assume it knows what it's doing!
- We don't review packaging changes - Let us know if you'd like us to change this.
|
PR Details of @2lambda123 in 3DUnetCNN :
|
PR summaryThis Pull Request, created automatically by Snyk, upgrades the SuggestionEnsure that the upgraded version of Disclaimer: This comment was entirely generated using AI. Be aware that the information provided may be incorrect. Current plan usage: 1.46% Have feedback or need help? |
penify-dev
bot
changed the title
scikit-learn version to fix vulnerability in legacy/requirements.txt
PR Review 🔍
|
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
|
|
Check out the playback for this Pull Request here. |
Micro-Learning Topic: Information disclosure (Detected by phrase)Matched on "information exposure"Many web applications and APIs do not properly protect sensitive data, such as financial, healthcare, and PII. Attackers may steal or modify such weakly protected data to conduct credit card fraud, identity theft, or other crimes. Sensitive data may be compromised without extra protection, such as encryption at rest or in transit, and requires special precautions when exchanged with the browser. Source: https://www.owasp.org/index.php/Category:OWASP_Top_Ten_Project Try a challenge in Secure Code WarriorMicro-Learning Topic: Sensitive information exposure (Detected by phrase)Matched on "Sensitive information exposure"Displaying too much information without proper access-control can lead to sensitive data being revealed that could be of value to an attacker directly or useful in a subsequent attack. Try a challenge in Secure Code WarriorHelpful references
|
PR Code Suggestions ✨
|
User description
This PR was automatically created by Snyk using the credentials of a real user.
Snyk has created this PR to fix 1 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):
legacy/requirements.txt⚠️ Warning
``` nilearn 0.10.2 requires scipy, which is not installed. nilearn 0.10.2 requires scikit-learn, which is not installed.