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

Added Environment Files #71

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Added Environment Files #71

wants to merge 6 commits into from

Conversation

srivarra
Copy link
Contributor

@srivarra srivarra commented Sep 28, 2023

  • added environment.yml files for macOS and Linux
  • removed notebook output

If you haven't already, please read through our contributing guidelines before opening your PR

What is the purpose of this PR?

Added an environment.yml file for Linux with CUDA support, and a macOS environment with Metal support.

How did you implement your changes

Bumped tensorflow to 2.13.
Removed the function normalize_data_format.

Remaining issues

  • macOS environment file successfully creates the environment
  • Linux environment file successfully creates the environment on Linux
  • Linux environment file successfully creates the environment on WSL2
  • macOS Tests pass
  • Linux Tests pass

@JLrumberger
Copy link
Collaborator

Hi @srivarra,
what's the status on this, did you uncover some major roadblocks?

@srivarra
Copy link
Contributor Author

srivarra commented Oct 6, 2023

@JLrumberger Yeah there are many issues with more recent versions of TensorFlow and Keras having incompatibilities. For example, some functions in Keras 2 (included in TF 2) are not available in newer TF versions, nor are they available in Keras 3.

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.

Add conda environment configurations for Linux w/ CUDA and macOS w/ Metal
2 participants