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.

Sign up for GitHub

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

Jump to bottom

Optimize constraints #54

Open
Enchufa2 opened this issue Nov 15, 2023 · 1 comment
Open

Optimize constraints #54

Enchufa2 opened this issue Nov 15, 2023 · 1 comment
Labels
optimization Code optimizations prio:low Priority Low

Comments

@Enchufa2
Copy link
Member

Enchufa2 commented Nov 15, 2023 •
edited
Loading

Currently, the keras and torch constraints are expensive due to the back and forth of the weights between the native and R representations. In fact, keras shows a warning saying that the callback takes too much time compared with the rest. We should see if there's a better way to implement this without bringing the matrices to R.
@Enchufa2 Enchufa2 added prio:low Priority Low optimization Code optimizations labels Nov 15, 2023
@Enchufa2
Copy link
Member Author

See: rstudio/keras3#1327

@Enchufa2 Enchufa2 mentioned this issue Nov 23, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
optimization Code optimizations prio:low Priority Low
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Enchufa2