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

Updated Requirement Issues #9

Merged
merged 10 commits into from
Sep 13, 2024
Merged

Updated Requirement Issues #9

merged 10 commits into from
Sep 13, 2024

Conversation

tanzim10
Copy link
Contributor

  • Updated torch from 1.3.1 to 2.0.0 to address compatibility issues with Python 3.9.
  • Updated scikit-learn from 1.2.1 to 1.5.1 to address compatibility issues with Python 3.9.
  • Adjusted numpy and pandas versions to align with the new torch requirements.
  • Added constraints to requirements.txt to prevent future version conflict issues.

Copy link

linux-foundation-easycla bot commented Sep 10, 2024

CLA Not Signed

@tanzim10
Copy link
Contributor Author

  • Updated the CCO Notebook to fix a bug cauased by this error " ValueError: Cannot subset columns with a tuple with more than one element. Use a list instead."
  • Fixed this issue by using an list for the rf predictions by replacing the current code with pd.concat([df1, df2]).groupby(["loc_x", "loc_y"], as_index=False)[["pred_rsrp", "cell_rxpwr_dbm"]].max()

Copy link
Collaborator

@paulvarkey paulvarkey left a comment

Choose a reason for hiding this comment

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

CI passes with these upgrades.

@paulvarkey paulvarkey merged commit 2057ab5 into lf-connectivity:main Sep 13, 2024
2 of 3 checks passed
@tanzim10 tanzim10 deleted the tanzim-patch-1 branch October 1, 2024 05:49
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.

3 participants