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

Versions of libraries in the requirements.txt #22

Open
xwjabc opened this issue Nov 24, 2022 · 1 comment
Open

Versions of libraries in the requirements.txt #22

xwjabc opened this issue Nov 24, 2022 · 1 comment

Comments

@xwjabc
Copy link

xwjabc commented Nov 24, 2022

Hi, I would like to know if the versions of libraries in the requirements.txt can be provided. Currently I am using the latest TF (2.11) that seems to have breaking change in the optimizer. Thus, the initialization for base class in AdamWeightDecay() (https://github.com/google-research/pix2seq/blob/main/models/model_utils.py#L104) has incorrect arguments compared to the corresponding ones in https://github.com/keras-team/keras/blob/v2.11.0/keras/optimizers/optimizer_experimental/adam.py#L86. Thanks!

@chentingpc
Copy link
Collaborator

I think the code should be compatible with latest (stable) version of tensorflow. AdamWeightDecay is a subclass of the default Adam optimizer in keras, so it is fine to have different input arguments for AdamWeightDecay as long as that for Adam is consistent (which it is).

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