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

remove unused code (tensor.py-> class tensor), remove old tests, add new tests #2311

Merged
merged 4 commits into from
Sep 17, 2024
Merged

remove unused code (tensor.py-> class tensor), remove old tests, add new tests #2311

merged 4 commits into from
Sep 17, 2024

Conversation

roman-opentensor
Copy link
Contributor

No description provided.

@roman-opentensor roman-opentensor self-assigned this Sep 17, 2024
@roman-opentensor roman-opentensor requested a review from a team September 17, 2024 04:18
thewhaleking
thewhaleking reviewed Sep 17, 2024
View reviewed changes
Copy link
Contributor

@thewhaleking thewhaleking left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we really want to remove all these tests? It seems like they can just be slightly modified to work with Tensor

@roman-opentensor
Copy link
Contributor Author

Do we really want to remove all these tests? It seems like they can just be slightly modified to work with Tensor

Fixed

@roman-opentensor roman-opentensor mentioned this pull request Sep 17, 2024
Closed
@roman-opentensor roman-opentensor merged commit e8ab3b9 into btsdk Sep 17, 2024
12 checks passed
@roman-opentensor roman-opentensor deleted the feat/roman/remove-class-tensor branch September 17, 2024 22:06
@ibraheem-opentensor ibraheem-opentensor mentioned this pull request Sep 25, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thewhaleking thewhaleking thewhaleking approved these changes

Assignees

@roman-opentensor roman-opentensor

Labels
bittensor testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@roman-opentensor @thewhaleking