Skip to content

Add Constraints #197

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

Closed
wants to merge 7 commits into from
Closed

Conversation

JimClarke5
Copy link
Contributor

Added Constraints classes.

I could not do RadialConstraint because it requires lambda and while_loop support.

Sync with master tensorflow on upstream
Merge main  branch to local branch
Update after losses merge
Change float attributes to double
@karllessard
Copy link
Collaborator

karllessard commented Feb 11, 2021

Hi @JimClarke5 , looks like there is now a conflict in this PR. Also, is there any preference of order for review between this PR and the Regularizers #198 ?

Oh, and there is also Learning Rates #106 , which has been sitting there for awhile now

@JimClarke5
Copy link
Contributor Author

JimClarke5 commented Feb 11, 2021

@karllessard Constraints and Regularizers should be independent of each other and all other framework packages. I’ll check the conflict.

@JimClarke5
Copy link
Contributor Author

Closing this PT to link to a new branch built off the latest master checkin.

@JimClarke5 JimClarke5 deleted the Constraints branch May 2, 2021 15:10
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.

2 participants