Skip to content

Makefile: fix fix-pip rule for Tensorflow#115

Merged
kba merged 1 commit intoOCR-D:masterfrom
bertsky:fix-fix-pip
Jun 18, 2020
Merged

Makefile: fix fix-pip rule for Tensorflow#115
kba merged 1 commit intoOCR-D:masterfrom
bertsky:fix-fix-pip

Conversation

@bertsky
Copy link
Copy Markdown
Collaborator

@bertsky bertsky commented Jun 18, 2020

(old recipe does not work anymore, we now
have TF2 as tensorflow and TF1 as tensorflow-gpu)

Fix #114

(old recipe does not work anymore, we now
 have TF2 as `tensorflow` and TF1 as `tensorflow-gpu`)
@bertsky
Copy link
Copy Markdown
Collaborator Author

bertsky commented Jun 18, 2020

@kba if CI completes, I recommend releasing as another minor of yesterday's version, i.e. v2020-06-17.2, because no modules changed

@bertsky bertsky requested a review from stweil June 18, 2020 12:09
@kba
Copy link
Copy Markdown
Member

kba commented Jun 18, 2020

@kba if CI completes, I recommend releasing as another minor of yesterday's version, i.e. v2020-06-17.2, because no modules changed

Sure. I'll also merge #103 then, okay?

@bertsky
Copy link
Copy Markdown
Collaborator Author

bertsky commented Jun 18, 2020

Sure. I'll also merge #103 then, okay?

Can you give me a few minutes for review?

@kba
Copy link
Copy Markdown
Member

kba commented Jun 18, 2020

Can you give me a few minutes for review?

Sure. #103 should have at least some documentation before merging tbh. I'll release the fix-pip fix first so we have working containers again soon.

@kba kba merged commit c2bf378 into OCR-D:master Jun 18, 2020
@kba
Copy link
Copy Markdown
Member

kba commented Jun 18, 2020

Released. However I'm worried that the version logic for ./release.sh release-dockerhub is flawed, I think the release will be v2020-06-18 on DockerHub, not v2020-06-17.2. If that happens, I have an idea how to fix.

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.

[Docker] Tensorflow conflict

2 participants