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

Upgrade Tensorflow to 2.4.1 #42

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

bkirwi
Copy link
Contributor

@bkirwi bkirwi commented Mar 10, 2021

What it says on the tin, basically.

There are more changes that this upgrade unlocks, like exposing some of the new methods and functionality in the 2.x series. But upgrading the version seems like enough for one PR!

I'll make a PR for the downloads as well and link it below.

@bkirwi
Copy link
Contributor Author

bkirwi commented Jul 16, 2021

@boncheolgu - what are the odds on getting a change like this merged? Wondering if I should resolve the conflict.

@boncheolgu
Copy link
Owner

@boncheolgu - what are the odds on getting a change like this merged? Wondering if I should resolve the conflict.

Contribution is always welcome. I will merge this if it passed CI.

@bkirwi
Copy link
Contributor Author

bkirwi commented Jan 21, 2022

@boncheolgu - apologies for letting this one sit for so long.

It appears that the issue was the includes list in Cargo.toml, which I've rewritten, and have confirmed that cargo package now works and the artifact still fits in the 10MB limit.

@bugeats
Copy link

bugeats commented Aug 1, 2022

Hey guys. What happened here? I would love to see this merged. Can we at least bump the build to see what the issue is? It looks like Github has removed the old build logs.

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.

3 participants