Remove tf.ENV

[nsthorat]

tf.ENV is a global mutable object that is exported from the module. This isn't allowed in ES2015 semantics and thus closure has problems with it.

Let's remove this in favor of tf.environment().

[rthadur]

@nsthorat i see the PR got merged , can we close this issue or is there more request coming through ?

[nsthorat]

The PR that got merged is just introducing tf.env(), we won't be able to remove tf.ENV until we want to make a breaking change for TensorFlow.js 2.0 so let's keep this open.

[rthadur]

sure , thanks for confirming

[gaikwadrahul8]

Hi, @nsthorat

Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.

The TFJs team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TFJs version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.

Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow.js space.

Thank you for your support and cooperation.

[github-actions]

This issue has been marked stale because it has no recent activity since 7 days. It will be closed if no further activity occurs. Thank you.

[github-actions]

This issue was closed due to lack of activity after being marked stale for past 7 days.

[google-ml-butler]

Are you satisfied with the resolution of your issue?
Yes
No