Skip to content

Bug Fixing Labs for DL #99

Merged
danielrazavi merged 6 commits intomainfrom
environment-update
Feb 9, 2026
Merged

Bug Fixing Labs for DL #99
danielrazavi merged 6 commits intomainfrom
environment-update

Conversation

@upadhyan
Copy link
Collaborator

@upadhyan upadhyan commented Feb 9, 2026

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

Updating pyproject.toml to handle dependency conflicts + disabling GPU for tensorflow across all labs. Specifically updated UMAP and sklearn versions so they install correctly. Furthermore, I'm adding a code block to disable GPU if the participant has a GPU on their machine, along w/ comments saying they can disable it if they want.

NOTES ON GPU DISABLING: Keras/Tensorflow often has problems if someone hasn't install ptxas, the proper cuda versions, updated drivers, etc. Its easier and more practical to just add a code bit to disable GPU access unless someone's confident in enabling it.

FUTURE WORK: Tensorflow/keras is cool, but its so clunky. Maybe we can go full torch/skorch (https://github.com/skorch-dev/skorch) after this?

Checklist

  • [ x] I can confirm that my changes are working as intended

@upadhyan upadhyan requested a review from danielrazavi February 9, 2026 05:10
@github-actions
Copy link

github-actions bot commented Feb 9, 2026

Hello, thank you for your contribution. If you are a participant, please close this pull request and open it in your own forked repository instead of here. Please read the instructions on your onboarding Assignment Submission Guide more carefully. If you are not a participant, please give us up to 72 hours to review your PR. Alternatively, you can reach out to us directly to expedite the review process.

Copy link
Member

@danielrazavi danielrazavi left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thank you.

@danielrazavi danielrazavi merged commit c108d06 into main Feb 9, 2026
1 check passed
@danielrazavi danielrazavi deleted the environment-update branch February 9, 2026 17:45
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.

2 participants