René's URL Explorer Experiment


Title: Framework: Move Ops parameter to call method where possible · Issue #202 · tensorflow/java · GitHub

Open Graph Title: Framework: Move Ops parameter to call method where possible · Issue #202 · tensorflow/java

X Title: Framework: Move Ops parameter to call method where possible · Issue #202 · tensorflow/java

Description: I'd like to move the Ops parameters of framework classes to the call method, where possible. This is primarily for Kotlin interop, but has a few other benefits as well. It won't be possible for stateful classes (Metrics, Optimizers), but...

Open Graph Description: I'd like to move the Ops parameters of framework classes to the call method, where possible. This is primarily for Kotlin interop, but has a few other benefits as well. It won't be possible for sta...

X Description: I'd like to move the Ops parameters of framework classes to the call method, where possible. This is primarily for Kotlin interop, but has a few other benefits as well. It won't be possible...

Opengraph URL: https://github.com/tensorflow/java/issues/202

X: @github

direct link

Domain: github.com


Hey, it has json ld scripts:
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Framework: Move Ops parameter to call method where possible","articleBody":"I'd like to move the `Ops` parameters of framework classes to the call method, where possible.  This is primarily for Kotlin interop, but has a few other benefits as well.  It won't be possible for stateful classes (Metrics, Optimizers), but should be possible for most, as far as I can tell (Initializers, Activations, Losses).  I'm going to use Losses as a standin for all 3 for my examples.\r\n1. Kotlin interop w/ `@FunctionInterface`.  If we do this, we can then define new losses like `Loss{ tf, x -\u003e tf.stuff(x) }` or pass lambdas to methods that take losses.  This is very nice for layers, where we might have something Keras-like `Layer(activation=ReLU())` but want to replace it with something custom.\r\n2. Re-use of objects.  Currently, losses create any subsequent calls in the same scope as their first call.  That means if it's called inside a sub-scope, including device ones, it ignores the scope.  This is somewhat expected, but not ideal.  It also causes further issues if we use `Ops` for (eager) tensor lifetime management, which has been suggested and is something I'd like to do (it's easy enough to make a long-lived copy of the initial scope, but then the tensors created in the call methods live forever, and the framework classes need to be closable).\r\n3. Passing configs, like Keras.  In Keras, if you have a activation or loss that requires some parameters, you can pass it to a layer like `Layer(activation=LeakyReLU(alpha=0.3))`.  I expect this will be common with our API, as well.  Currently, this runs into the above issue w/ scoping, and prevents you from passing activations (or losses) from scopes that don't have an Ops instance available.\r\n\r\nI'd look at having the stateful classes take `Ops` in `call` as well, and only using the constructor ops for initializing state.  This works better with scoping and lifetimes as mentioned above.","author":{"url":"https://github.com/rnett","@type":"Person","name":"rnett"},"datePublished":"2021-01-31T04:10:45.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":25},"url":"https://github.com/202/java/issues/202"}

route-pattern/_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format)
route-controllervoltron_issues_fragments
route-actionissue_layout
fetch-noncev2:5b78722e-ac27-433c-d961-5a95b842715f
current-catalog-service-hash81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114
request-idE75C:392B41:BD38F9:1043490:696A8922
html-safe-nonce9b9ccb06af54a0e7ee2b31e20356c05294c75805329927bd10b2992e85112388
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFNzVDOjM5MkI0MTpCRDM4Rjk6MTA0MzQ5MDo2OTZBODkyMiIsInZpc2l0b3JfaWQiOiI3ODAzOTg1NjgyMDI2MzY3MjY2IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0=
visitor-hmacd1d928c072a69e5c55327fb75151c3d3c59d5ebe9f97e84ee1607e5150237c17
hovercard-subject-tagissue:797613515
github-keyboard-shortcutsrepository,issues,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location///voltron/issues_fragments/issue_layout
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/tensorflow/java/202/issue_layout
twitter:imagehttps://opengraph.githubassets.com/4263a9fa9ea0dcc73257c5113788873efcc86f51ea1daf1c56209619eef490a9/tensorflow/java/issues/202
twitter:cardsummary_large_image
og:imagehttps://opengraph.githubassets.com/4263a9fa9ea0dcc73257c5113788873efcc86f51ea1daf1c56209619eef490a9/tensorflow/java/issues/202
og:image:altI'd like to move the Ops parameters of framework classes to the call method, where possible. This is primarily for Kotlin interop, but has a few other benefits as well. It won't be possible for sta...
og:image:width1200
og:image:height600
og:site_nameGitHub
og:typeobject
og:author:usernamernett
hostnamegithub.com
expected-hostnamegithub.com
Noned2aa6dedd7d97fa651b17c40401f2605f309df5ca5fb659bf0554cf34f1de825
turbo-cache-controlno-preview
go-importgithub.com/tensorflow/java git https://github.com/tensorflow/java.git
octolytics-dimension-user_id15658638
octolytics-dimension-user_logintensorflow
octolytics-dimension-repository_id207384523
octolytics-dimension-repository_nwotensorflow/java
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forkfalse
octolytics-dimension-repository_network_root_id207384523
octolytics-dimension-repository_network_root_nwotensorflow/java
turbo-body-classeslogged-out env-production page-responsive
disable-turbofalse
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
releasec1dbca8981c5e3d6a69bce2f1d289e732b86241d
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://github.com/tensorflow/java/issues/202#start-of-content
https://github.com/
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Ftensorflow%2Fjava%2Fissues%2F202
GitHub CopilotWrite better code with AIhttps://github.com/features/copilot
GitHub SparkBuild and deploy intelligent appshttps://github.com/features/spark
GitHub ModelsManage and compare promptshttps://github.com/features/models
MCP RegistryNewIntegrate external toolshttps://github.com/mcp
ActionsAutomate any workflowhttps://github.com/features/actions
CodespacesInstant dev environmentshttps://github.com/features/codespaces
IssuesPlan and track workhttps://github.com/features/issues
Code ReviewManage code changeshttps://github.com/features/code-review
GitHub Advanced SecurityFind and fix vulnerabilitieshttps://github.com/security/advanced-security
Code securitySecure your code as you buildhttps://github.com/security/advanced-security/code-security
Secret protectionStop leaks before they starthttps://github.com/security/advanced-security/secret-protection
Why GitHubhttps://github.com/why-github
Documentationhttps://docs.github.com
Bloghttps://github.blog
Changeloghttps://github.blog/changelog
Marketplacehttps://github.com/marketplace
View all featureshttps://github.com/features
Enterpriseshttps://github.com/enterprise
Small and medium teamshttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
App Modernizationhttps://github.com/solutions/use-case/app-modernization
DevSecOpshttps://github.com/solutions/use-case/devsecops
DevOpshttps://github.com/solutions/use-case/devops
CI/CDhttps://github.com/solutions/use-case/ci-cd
View all use caseshttps://github.com/solutions/use-case
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
View all industrieshttps://github.com/solutions/industry
View all solutionshttps://github.com/solutions
AIhttps://github.com/resources/articles?topic=ai
Software Developmenthttps://github.com/resources/articles?topic=software-development
DevOpshttps://github.com/resources/articles?topic=devops
Securityhttps://github.com/resources/articles?topic=security
View all topicshttps://github.com/resources/articles
Customer storieshttps://github.com/customer-stories
Events & webinarshttps://github.com/resources/events
Ebooks & reportshttps://github.com/resources/whitepapers
Business insightshttps://github.com/solutions/executive-insights
GitHub Skillshttps://skills.github.com
Documentationhttps://docs.github.com
Customer supporthttps://support.github.com
Community forumhttps://github.com/orgs/community/discussions
Trust centerhttps://github.com/trust-center
Partnershttps://github.com/partners
GitHub SponsorsFund open source developershttps://github.com/sponsors
Security Labhttps://securitylab.github.com
Maintainer Communityhttps://maintainers.github.com
Acceleratorhttps://github.com/accelerator
Archive Programhttps://archiveprogram.github.com
Topicshttps://github.com/topics
Trendinghttps://github.com/trending
Collectionshttps://github.com/collections
Enterprise platformAI-powered developer platformhttps://github.com/enterprise
GitHub Advanced SecurityEnterprise-grade security featureshttps://github.com/security/advanced-security
Copilot for BusinessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/premium-support
Pricinghttps://github.com/pricing
Search syntax tipshttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
documentationhttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Ftensorflow%2Fjava%2Fissues%2F202
Sign up https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fvoltron%2Fissues_fragments%2Fissue_layout&source=header-repo&source_repo=tensorflow%2Fjava
Reloadhttps://github.com/tensorflow/java/issues/202
Reloadhttps://github.com/tensorflow/java/issues/202
Reloadhttps://github.com/tensorflow/java/issues/202
tensorflow https://github.com/tensorflow
javahttps://github.com/tensorflow/java
Notifications https://github.com/login?return_to=%2Ftensorflow%2Fjava
Fork 223 https://github.com/login?return_to=%2Ftensorflow%2Fjava
Star 913 https://github.com/login?return_to=%2Ftensorflow%2Fjava
Code https://github.com/tensorflow/java
Issues 151 https://github.com/tensorflow/java/issues
Pull requests 23 https://github.com/tensorflow/java/pulls
Actions https://github.com/tensorflow/java/actions
Projects 0 https://github.com/tensorflow/java/projects
Security Uh oh! There was an error while loading. Please reload this page. https://github.com/tensorflow/java/security
Please reload this pagehttps://github.com/tensorflow/java/issues/202
Insights https://github.com/tensorflow/java/pulse
Code https://github.com/tensorflow/java
Issues https://github.com/tensorflow/java/issues
Pull requests https://github.com/tensorflow/java/pulls
Actions https://github.com/tensorflow/java/actions
Projects https://github.com/tensorflow/java/projects
Security https://github.com/tensorflow/java/security
Insights https://github.com/tensorflow/java/pulse
New issuehttps://github.com/login?return_to=https://github.com/tensorflow/java/issues/202
New issuehttps://github.com/login?return_to=https://github.com/tensorflow/java/issues/202
Framework: Move Ops parameter to call method where possiblehttps://github.com/tensorflow/java/issues/202#top
https://github.com/rnett
https://github.com/rnett
rnetthttps://github.com/rnett
on Jan 31, 2021https://github.com/tensorflow/java/issues/202#issue-797613515
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width


URLs of crawlers that visited me.