Skip to content

Made the classifier work in a .ipynb notebook#695

Open
00tanveer wants to merge 1 commit intofastai:masterfrom
00tanveer:master
Open

Made the classifier work in a .ipynb notebook#695
00tanveer wants to merge 1 commit intofastai:masterfrom
00tanveer:master

Conversation

@00tanveer
Copy link
Copy Markdown

No description provided.

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@ashwin-r11
Copy link
Copy Markdown

Hi @00tanveer,
how was it possible to introduce nearly 11,500 inclusions and 1,000 exclusions in the codebase within a single commit?

Was this generated via vibe coding? If so, please verify the code integrity and structure carefully before raising a PR. LLM-generated code requires extensive review prior to merging, as it is more prone to logical issues and security vulnerabilities.

Vibe coding is fine, but it is essential to manually review, test, and validate all AI-assisted code before pushing. Also, please split changes into logical, auditable commits with meaningful messages. Ideally, each commit should contain ~100–500 lines of changes, unless there is a strong justification otherwise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants