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

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

Merged
merged 50 commits into from
Aug 6, 2024
Merged

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

merged 50 commits into from
Aug 6, 2024

Conversation

roman-opentensor
Copy link
Contributor

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

@gus-opentensor gus-opentensor gus-opentensor approved these changes

Assignees

@roman-opentensor roman-opentensor

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
@roman-opentensor @gus-opentensor