Sitelet https://github.com/tensorflow/tensorflow/issues/25067
Skip to content

Latest commit in deprecation.py breaks R TensorFlow client  #25067

Description

@skeydan

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template

System information

  • Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
  • OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Fedora 28
  • Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: NA
  • TensorFlow installed from (source or binary): binary
  • TensorFlow version (use command below): nightly (r1.13)
  • Python version: 3.6
  • Bazel version (if compiling from source): NA
  • GCC/Compiler version (if compiling from source): NA
  • CUDA/cuDNN version: NA
  • GPU model and memory: NA

Hi,

the commit

b97727b#diff-fa5fb3b8d9f512ad269f7eb67903ec2b

breaks the R TensorFlow client which uses embedded Python from R (see rstudio/tfdatasets#17)

Specifically, the line

frame = stack[-4 if outer else -3]

triggers the error

Error in py_call_impl(callable, dots$args, dots$keywords) : 
  IndexError: list index out of range

Detailed traceback: 
  File "/home/key/anaconda3/envs/tf-master-1215/lib/python3.6/site-packages/tensorflow/python/util/deprecation.py", line 314, in new_func
    _call_location(), decorator_utils.get_qualified_name(func),
  File "/home/key/anaconda3/envs/tf-master-1215/lib/python3.6/site-packages/tensorflow/python/util/deprecation.py", line 103, in _call_location
    frame = stack[-4 if outer else -3]

on our side. For us, in one specific example, instead of a list with at least 4 items, stack contains a list of 2 tuples, each of which are of length 6.

Unfortunately, this will error and stop execution every time a new deprecation warning is issued.

Current workaround on our side is rstudio/tensorflow#287, however this comes at the cost of totally disabling deprecation warnings.

Is there anything you could do to make this change compatible?

Thanks,
Sigrid

No activity

Activity on this issue will appear here.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

TF 2.0Issues relating to TensorFlow 2.0stat:awaiting tensorflowerStatus - Awaiting response from tensorflowertype:bugBug

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions