Skip to content
This repository has been archived by the owner on Mar 17, 2021. It is now read-only.

Modified tests to workaround TF #29439 issue and checked the compatibility with TF 1.13.1 and 1.14.0rc0 #411

Merged
merged 3 commits into from Jun 29, 2019

Conversation

danieltudosiu
Copy link
Contributor

Modified the tests as a workaround the TF #29439 issue and modified t…he crf.py and misc_io.py to work with TF 1.13.1 and 1.14.0rc0.

Status

READY

Description

The aim of the branch was to solve the TF #29439 issue which was done by creating our own testing case class which inherits from the TF one and deletes the test_case method (bad oop design, but temporary).

Also, the codebase was tested against TF 1.13.1 and 1.14.0rc0 and the required changes to work were made.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • My code follows the code style of this project.
  • All new and existing tests passed.

Impacted Areas in Application

Nearly all tests
CRF Layer
misc_io.py

@ericspod
Copy link
Collaborator

It's failing 2 of the early stopping tests. @tomvars any idea?

@csudre csudre merged commit e6150ec into dev Jun 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants