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.

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

fix: adds versions to requirements.txt #165

Merged
merged 3 commits into from
Jul 31, 2024

Conversation

grantdfoster
Copy link
Contributor

@grantdfoster grantdfoster commented Jul 24, 2024

Description

This PR fixes #144. It adds to the requirements.txt file to specify exact versions for project dependencies:

  • bittensor==7.1.0
  • loguru==0.7.2
  • python-dotenv==0.21.0
  • torch==2.3.0
  • watchfiles==0.22.0

Why This Is Needed

  1. Consistency: Ensures all environments use the same dependency versions.
  2. Reproducibility: Enables consistent builds and behavior across different setups.
  3. Stability: Prevents breaking changes from newer versions, maintaining project stability.

How It Ensures Stability

  1. Deterministic Builds: Guarantees the same set of dependencies is installed every time.
  2. Controlled Upgrades: Allows systematic testing before upgrading to newer versions.
  3. Conflict Avoidance: Reduces risk of dependency conflicts.

Notes for Reviewers
None

Signed commits

  • Yes, I signed my commits.

@grantdfoster grantdfoster linked an issue Jul 24, 2024 that may be closed by this pull request
hide-on-bush-x
hide-on-bush-x previously approved these changes Jul 31, 2024
@grantdfoster grantdfoster merged commit ce2c875 into main Jul 31, 2024
3 checks passed
@grantdfoster grantdfoster deleted the fix--requirements-versioning branch July 31, 2024 20:37
grantdfoster added a commit that referenced this pull request Jul 31, 2024
* fix: adds versions to requirements.txt

* fix: update bittensor version

* fix: btcli
grantdfoster added a commit that referenced this pull request Aug 6, 2024
* fix(responses): getting entire synapse and surfacing processing times on validator

* feat: adds processing time to response logging

* fix: remove process times from get_rewards until data validation is implemented

* fix: adds latency to validator responses

* fix: better sorting on the validator response for now

* fix: removes double import

* fix: guards against deployed validators not correctly pinging uids

* fix: helper for sending TAO

* feat: responses classification

* feat: wip twitter endpoints set

* feat: logic fix

* chore: updates requirements.txt

* fix: adjust timeout on vali

* fix: points to protocol docs instead of duplicating (#163)

* fix: adds versions to requirements.txt (#165)

* fix: adds versions to requirements.txt

* fix: update bittensor version

* fix: btcli

* feat: version check guards on neurons, logging cleanup

* feat: miner version on vali exposed

* feat: begging of miner version call

* feat: miner versions recorded using PingMiner request

* fix: remove request type for version

* fix: sleep for a minute on miner version check

* fix: rename function to wrapper

---------

Co-authored-by: hide-on-bush-x <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update requirements.txt
2 participants