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

Compile cpu #48

Closed
wants to merge 6 commits into from
Closed

Compile cpu #48

wants to merge 6 commits into from

Conversation

bhack
Copy link
Contributor

@bhack bhack commented Nov 26, 2021

This is trying to figure out/monitor how much time, a build with the remote cache, requires on the standard Github Action hardware resources.

@bhack
Copy link
Contributor Author

bhack commented Nov 27, 2021

We have some issue to match the remote cache here:

[12,071 / 13,189] Github Action timeout after ~ 6 Hours running the compilation job.

@bhack
Copy link
Contributor Author

bhack commented Nov 30, 2021

As you seems not interested in #47 this is just the compile job in your published image on DockerHUb.

@angerson
Copy link
Contributor

angerson commented Dec 2, 2021

This is something we can revisit later (Q1+) once the needs for internal CI have been covered.

@bhack
Copy link
Contributor Author

bhack commented Dec 2, 2021

Ok but I don't know if the problem is at the CI level. it would be nice if you could compare the execution log with two builds on exactly same commit on the CI.

I've tried on the nightly commit and the level of cache misses is different in every nightly release:

https://discuss.tensorflow.org/t/adopting-open-source-dockerfiles-for-official-tf-nightly-ci/6050/17

@bhack
Copy link
Contributor Author

bhack commented Aug 1, 2022

/cc @theadactyl

@bhack
Copy link
Contributor Author

bhack commented Sep 6, 2022

As we have recently removed this file from the repository I have refactored this in TF with tensorflow/tensorflow#57630

@bhack bhack closed this Sep 6, 2022
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.

None yet

2 participants