Skip to content

Part 2: Redesign, fix namespace conflicts, remove btcli#2204

Merged
basfroman merged 50 commits intobtsdkfrom
feat/roman/btsdk-structure
Aug 6, 2024
Merged

Part 2: Redesign, fix namespace conflicts, remove btcli#2204
basfroman merged 50 commits intobtsdkfrom
feat/roman/btsdk-structure

Conversation

@basfroman
Copy link
Collaborator

@basfroman basfroman commented Aug 5, 2024

  • this PR prepared the SDK for refactoring the subtensor and extrinsics.
  • completely new package structure is used, but backward compatibility with bittensor version 7.3.0 and below is maintained (including all paths to subpackages).
  • the test "tests.unit_tests.test_axon.test_priority_pass" is skipped because for unknown (yet) reasons it enters an endless loop. It will be fixed in the next changes.

Next steps:

  • check all parts of the code for use in bittensor and subnets.
  • subtensor / extrinsics optimization (according to backward compatibility).

Roman added 30 commits July 29, 2024 16:43
Rebase to feat/roman/btsdk
0948bb4
Fix test
e0b3fae
Temporary placement of cli and related content in btcli (for manual t…
82a3e20 
…esting during development)
Move constants to their related modules.
ff54d57
Refactoring types.py
cce27b8
Refactoring and typos in errors.py
b61adb0
Refactoring metagraph.py
a82d38e
Refactoring metagraph.py
91df5eb
metagraph.py to bittensor/core
94b2be9
axon.py to bittensor/core
71f23fb
error.py and types.py to bittensor/core
c55dcad
subnets.py to bittensor/core
e9ce941
synapse.py to bittensor/core
006e1a5
tensor.py to bittensor/core
ccbc28d
stream.py to bittensor/core
1c4744a
threadpool.py to bittensor/core
a7a39b7
dendrite.py to bittensor/core
f660bad
chain_data.py to bittensor/core
9838ed3
settings.py to bittensor/core
9c952eb
create bittensor.api sub-package
c9e59a7
Move subtensor.py to bittensor.core and fix related test
e4c1ab4
Move bittensor.extrinsics to bittensor.api.extrinsics and fix rel…
13081bf 
…ated test.
Return config.py back to the bittensor
a23b861
Move constants from bt/__init__.py to bt/core/settings.py
30e639f
Fix order import problem for bt/__init__.py
087b1ff
Remove unused error parser based on metadata
702b74e
Move delegates_details_url from bt/__init__.py to settings.py
d69e6ed
Bump up setuptools version
d17fdad
Make version.py bittensor import independent
c047b48
Update setup.py, replace file with the __version__
a262dd0
Roman added 20 commits August 1, 2024 18:34
Update settings.py
1b7ceda
Make axon independent of bittensor import. Fix axon tests.
544c4b7
Make chain_data.py independent of bittensor import. Fix chain_data te…
316f7af 
…sts.
Make dendrite.py independent of bittensor import. Fix dendrite tests.
23ba67b
Make staking.py independent of bittensor import. Fix staking tests.
137b929
Make subtensor.py independent of bittensor import. Fix subtensor tests.
1b228ed
Make prometheus.py independent of bittensor import. Fix prometheus te…
ffa30b5 
…sts.
Axon refactoring
6bd0647
Move subnets.py into bittensor/utils since this module is not used wi…
856be50 
…thin the bittensor package, but is actively used by the community.
Make synapse.py independent of bittensor import. Fix synapse tests.
a859e94
Make tensor.py independent of bittensor import. Fix tensor tests.
dc394d8
Change name bittensor.utils.hash -> bittensor.utils.get_hash
6db9a38
Moved from using import bittensor (withing bittensor codebase) to d…
a23ff8c 
…irect imports of modules and other objects. It helped to avoid namespace conflicts.
Add backwards compatibility for 'bittensor.api.extrinsics' as 'bitten…
a95ee4c 
…sor.extrinsics'
Move the contents of wallet_utils.py to `bittensor/utils/__init__.p…
f68ed6e 
…y`, since it is used directly from utils.
Delete bccli staff and related tests. Fix setup.py.
3c6aa67
Fix tests.
a74c5c5
Makes the bittensor.utils.mock subpackage available as `bittensor.m…
f254172 
…ock` for backwards compatibility.
Check for participation before nomination call
f55982c
Ruff formatting. Fix ruff's complaints.
da89cd3
@basfroman basfroman self-assigned this Aug 5, 2024
@basfroman basfroman added enhancement New feature or request bittensor labels Aug 5, 2024
@basfroman basfroman requested a review from a team August 5, 2024 18:53
@basfroman basfroman mentioned this pull request Aug 6, 2024
Closed
8 tasks
@basfroman basfroman merged commit 4323e21 into btsdk Aug 6, 2024
@basfroman basfroman deleted the feat/roman/btsdk-structure branch August 6, 2024 19:43
@ibraheem-abe ibraheem-abe 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

1 more reviewer

@thealligatorking thealligatorking thealligatorking approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@basfroman basfroman

Labels

bittensor enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@basfroman @thealligatorking