Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Revert "Add LeNet Implementation in PyTorch" #8621

Merged
merged 1 commit into from
Apr 8, 2023
Merged

Revert "Add LeNet Implementation in PyTorch" #8621

merged 1 commit into from
Apr 8, 2023

Conversation

cclauss
Copy link
Member

@cclauss cclauss commented Apr 7, 2023

Reverts #7070

@algorithms-keeper algorithms-keeper bot added awaiting reviews This PR is ready to be reviewed enhancement This PR modified some existing files labels Apr 7, 2023
@cclauss cclauss enabled auto-merge (squash) April 7, 2023 20:02
@cclauss cclauss requested a review from dhruvmanila April 7, 2023 20:03
@tianyizheng02
Copy link
Contributor

Why revert the PR?

@cclauss
Copy link
Member Author

cclauss commented Apr 7, 2023

How-to-use that fails ci tests

@amirsoroush
Copy link
Contributor

amirsoroush commented Apr 7, 2023 •
edited
Loading

@tianyizheng02 After merging that PR, three subsequent PRs failed to pass tests. Seems like "ruff" complains about something.

@amirsoroush amirsoroush mentioned this pull request Apr 7, 2023
Merged
14 tasks
@cclauss cclauss merged commit 179298e into master Apr 8, 2023
@algorithms-keeper algorithms-keeper bot removed awaiting reviews This PR is ready to be reviewed labels Apr 8, 2023
@cclauss cclauss deleted the revert-7070-master branch April 8, 2023 01:56
tianyizheng02 pushed a commit to tianyizheng02/Python that referenced this pull request May 29, 2023
@cclauss @tianyizheng02
Revert "Add LeNet Implementation in PyTorch (TheAlgorithms#7070)" (Th…
b916450 
…eAlgorithms#8621)

This reverts commit b2b8585.
sedatguzelsemme pushed a commit to sedatguzelsemme/Python that referenced this pull request Sep 15, 2024
@cclauss @sedatguzelsemme
Revert "Add LeNet Implementation in PyTorch (TheAlgorithms#7070)" (Th…
e892665 
…eAlgorithms#8621)

This reverts commit b2b8585.
@isidroas isidroas mentioned this pull request Jan 25, 2025
Closed
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dhruvmanila dhruvmanila dhruvmanila approved these changes

Assignees
No one assigned
Labels
enhancement This PR modified some existing files
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@cclauss @tianyizheng02 @amirsoroush @dhruvmanila