Skip to content

Increase Bittensor SDK test coverage#2262

Merged
basfroman merged 19 commits intobtsdkfrom
tests/roman/utils-tests
Aug 26, 2024
Merged

Increase Bittensor SDK test coverage#2262
basfroman merged 19 commits intobtsdkfrom
tests/roman/utils-tests

Conversation

@basfroman
Copy link
Collaborator

No description provided.

@basfroman basfroman marked this pull request as ready for review August 22, 2024 21:04
@basfroman basfroman requested a review from a team August 22, 2024 22:17
@basfroman basfroman self-assigned this Aug 22, 2024
return info_dictionary


# Subnet 24 uses this function
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this needed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Is this needed?

I'd like to leave this here. I marked this function because it is not used in bittensor, but we should keep it because of subnet 24. Does it make sense?

@basfroman basfroman merged commit effb2d2 into btsdk Aug 26, 2024
@basfroman basfroman deleted the tests/roman/utils-tests branch August 26, 2024 15:27
@ibraheem-abe ibraheem-abe mentioned this pull request Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants