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

Minor fix for L0_backend_python #7274

Merged
merged 1 commit into from
May 28, 2024
Merged

Minor fix for L0_backend_python #7274

merged 1 commit into from
May 28, 2024

Conversation

krishung5
Copy link
Contributor

@krishung5 krishung5 commented May 25, 2024 •
edited
Loading

  • Fix the conda issue for Python 3.11
    • There are some dependency problems: UNAVAILABLE: Internal: ModuleNotFoundError: No module named 'six' even though six is installed already. Seems like using conda install for TF for Python 3.11 doesn't not mess up with conda env anymore.
  • Place the client log of L0_backend_python/example to the outer directory for CI to capture artifacts

@krishung5 krishung5 marked this pull request as ready for review May 25, 2024 00:06
jbkyang-nvi
jbkyang-nvi approved these changes May 28, 2024
View reviewed changes
Copy link
Contributor

@jbkyang-nvi jbkyang-nvi left a comment

Choose a reason for hiding this comment

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

🙏 thanks for the fix!

@krishung5 krishung5 merged commit 92c9e8c into main May 28, 2024
3 checks passed
@krishung5 krishung5 deleted the krish-fix-ci branch May 28, 2024 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jbkyang-nvi jbkyang-nvi jbkyang-nvi approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@krishung5 @jbkyang-nvi