Skip to content

Update to Tensorflow 2.10.0#43

Open
sumit0190 wants to merge 20 commits into
master-2.9.1-rebased-against-2.8.2from
master-2.10.0
Open

Update to Tensorflow 2.10.0#43
sumit0190 wants to merge 20 commits into
master-2.9.1-rebased-against-2.8.2from
master-2.10.0

Conversation

@sumit0190
Copy link
Copy Markdown

@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
Copy Markdown

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
Copy Markdown

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
Copy Markdown

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
Comment thread tensorflow-base-cpu-gpu-linwin/meta.yaml
Comment thread tensorflow-base-cpu-gpu-linwin/meta.yaml Outdated
Comment thread tensorflow-base-cpu-osx/meta.yaml
Comment thread tensorflow-estimator/meta.yaml
Copy link
Copy Markdown
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
Copy Markdown

@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
Copy Markdown
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