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

Update to Tensorflow 2.10.0 #43

Open
wants to merge 20 commits into
base: master-2.9.1-rebased-against-2.8.2
Choose a base branch
from

Conversation

sumit0190
Copy link

@sumit0190 sumit0190 commented Oct 19, 2022

Currently this is meant only for linux-64 (i.e., builds successfully and has been tested), other platforms to follow.

The compiler restriction is necessary - the GPU variant does not build with 11.2 which is pinned in the global cbc.yaml. The developers have no intention of supporting any version other than the one which is officially supported so there isn't a lot we can do.

@anaconda-pkg-build
Copy link

Linter check failed, with the following output:
The following problems have been found:

ERROR: /tmp/abs_a1ib4iz79g/clone/recipe/meta.yaml:0: missing_meta_yaml: The recipe is missing a meta.yaml file
Errors were found

2 similar comments
@anaconda-pkg-build
Copy link

Linter check failed, with the following output:
The following problems have been found:

ERROR: /tmp/abs_a1ib4iz79g/clone/recipe/meta.yaml:0: missing_meta_yaml: The recipe is missing a meta.yaml file
Errors were found

@anaconda-pkg-build
Copy link

Linter check failed, with the following output:
The following problems have been found:

ERROR: /tmp/abs_a1ib4iz79g/clone/recipe/meta.yaml:0: missing_meta_yaml: The recipe is missing a meta.yaml file
Errors were found

@sumit0190 sumit0190 changed the base branch from master to master-2.9.1-rebased-against-2.8.2 October 27, 2022 12:17
@sumit0190 sumit0190 requested a review from katietz October 27, 2022 12:20
tensorflow-base-cpu-gpu-linwin/meta.yaml Show resolved Hide resolved
tensorflow-base-cpu-gpu-linwin/meta.yaml Outdated Show resolved Hide resolved
tensorflow-base-cpu-osx/meta.yaml Show resolved Hide resolved
tensorflow-estimator/meta.yaml Show resolved Hide resolved
Copy link
Contributor

@katietz katietz left a comment

Choose a reason for hiding this comment

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

lgtm, thanks

@varlackc varlackc self-requested a review October 27, 2022 16:18
Copy link

@varlackc varlackc left a comment

Choose a reason for hiding this comment

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

The build seems to be failing in many different architectures.
We might have to double check how to resolve the issue before merging.

@sumit0190
Copy link
Author

@varlackc Yeah Prefect can't build any recipe under this folder because it's not a "feedstock", it's just a collection of recipes. For now we can only build these recipes manually on our builders or dev instances.

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.

5 participants