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

the val_loss can't converge in tensorflow 1.15.2, while tf 1.2.1 works #32

Open
Tangzy7 opened this issue Mar 5, 2020 · 3 comments
Open

Comments

@Tangzy7
Copy link

Tangzy7 commented Mar 5, 2020

Running train.py with Tensorflow 1.15.2 in 20news, I got the val_loss 0.6x, while the training loss got 0.0x
However, the problem won't happen in tensorflow 1.2.1. Other dependencies are same with requirement.txt

Would you mind if you check this problem?

@hugochan
Copy link
Owner

hugochan commented Mar 5, 2020

Running train.py with Tensorflow 1.15.2 in 20news, I got the val_loss 0.6x, while the training loss got 0.0x
However, the problem won't happen in tensorflow 1.2.1. Other dependencies are same with requirement.txt

Would you mind if you check this problem?

It might be because Tensorflow 1.15.2 made some important changes which actually made some difference here. It's been a long time since the release of TensorFlow 1.2.1. Can you please share what changes you made to the original code in order to adapt it to TensorFlow 1.15.2? You might also want to check out #24.

@Tangzy7
Copy link
Author

Tangzy7 commented Mar 9, 2020

Running train.py with Tensorflow 1.15.2 in 20news, I got the val_loss 0.6x, while the training loss got 0.0x
However, the problem won't happen in tensorflow 1.2.1. Other dependencies are same with requirement.txt
Would you mind if you check this problem?

It might be because Tensorflow 1.15.2 made some important changes which actually made some difference here. It's been a long time since the release of TensorFlow 1.2.1. Can you please share what changes you made to the original code in order to adapt it to TensorFlow 1.15.2? You might also want to check out #24.

I just change some codes to adapt python3, while made no changes to the model. I just saw the requirements.txt said to use tensorflow 1.15.2.

@hugochan
Copy link
Owner

Running train.py with Tensorflow 1.15.2 in 20news, I got the val_loss 0.6x, while the training loss got 0.0x
However, the problem won't happen in tensorflow 1.2.1. Other dependencies are same with requirement.txt
Would you mind if you check this problem?

It might be because Tensorflow 1.15.2 made some important changes which actually made some difference here. It's been a long time since the release of TensorFlow 1.2.1. Can you please share what changes you made to the original code in order to adapt it to TensorFlow 1.15.2? You might also want to check out #24.

I just change some codes to adapt python3, while made no changes to the model. I just saw the requirements.txt said to use tensorflow 1.15.2.

Hi, the original code was tested under Tensorflow 1.2.1. I changed it to Tensorflow = 1.15.2 in requirements.txt as suggested by Github due to the security issue. Can you try Tensorflow 1.2.1 first to see if it works?

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

No branches or pull requests

2 participants