Change visibility status for schema_conversion_utils to allow compila…
@majiddadashi @gbaned it looks like http://ml-ci.amd.com:21096/job/tensorflow/job/github-prs-upstream-master/job/AMD-ROCm-Community-CI-Build/job/PR-63075/1/display/redirect fail is unrelated?
Do you have suggestions to get this PR approved? > @majiddadashi @gbaned it looks like http://ml-ci.amd.com:21096/job/tensorflow/job/github-prs-upstream-master/job/AMD-ROCm-Community-CI-Build/job/PR-63075/1/display/redirect fail is unrelated? Do you have suggestions to get this PR approved?

A similar situation appeared in [a recent PR](https://github.com/tensorflow/tensorflow/pull/63049) (unrelated failure for AMD-ROCm), but was pushed through. @mihaimaruseac was instrumental to have this approved... @mihaimaruseac look like merging is still blocked, could you advice?
 Sorry, this was stuck on internal review (see the image above https://github.com/tensorflow/tensorflow/blob/master/CONTRIBUTING.md#contributor-license-agreements for the process).

There is one internal comment that was not transferred here:

> the visibility of this should not be change, rather the header should be included in the target tflite_internal_cc_3p_api_deps_src", Thanks, @mihaimaruseac. @Namburger, can you please provide more insight/suggestions on how to implement the requested change? Is this on the TF or on the libcoral side?
 @mihaimaruseac, I manually changed the definition of `tflite_internal_cc_3p_api_deps_src` in `tensorflow/lite/schema/BUILD` as suggested:

```
filegroup(
    name = "tflite_internal_cc_3p_api_deps_src",
    srcs = [
        ":schema_fbs_srcs",
        ":schema_utils.cc",
        ":schema_utils.h",
        ":schema_conversion_utils.cc",
        ":schema_conversion_utils.h",
        
    ],
    visibility = [
        "//tensorflow/lite:__pkg__",
    ],
)
```

Yet compilation of libcoral stops with the same error.  @feranick can you post the log failures? > filegroup(
>     name = "tflite_internal_cc_3p_api_deps_src",
>     srcs = [
>         ":schema_fbs_srcs",
>         ":schema_utils.cc",
>         ":schema_utils.h",
>         ":schema_conversion_utils.cc",
>         ":schema_conversion_utils.h",
>         
>     ],
>     visibility = [
>         "//tensorflow/lite:__pkg__",
>     ],
> )

Here it is, both the procedure to reproduce and the log.

[procedure.txt](https://github.com/tensorflow/tensorflow/files/14488001/procedure.txt)
[log.txt](https://github.com/tensorflow/tensorflow/files/14488003/log.txt)

This is the identical error and log you would get from an untouched TF repo.
 I am going to close this as it is resolved internally with commit https://github.com/tensorflow/tensorflow/commit/79ecb3f8bb6bd73f0115fa9a97b630a6f745a426
…tion of ext. libraries

When compiling libraries such as [libcoral](https://github.com/google-coral/libcoral) and [pycoral](https://github.com/google-coral/pycoral), they are linked to TF and access header files in TF proper. One such case is [schema_conversion_utils.h](https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/schema/schema_conversion_utils.h). Unfortunately the visibility of such header is set in [BUILD, line 188](https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/schema/BUILD) as:

visibility = [":utils_friends"],

This prevents a successful compilation of the libraries (especially when compiled within a docker), as the libraries are not part of the `utils_friends`. It is notable that a similar issue (i.e. same visibility status) was present in TF2.15 for [schema_util.h](https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/schema/schema_util.h), but was recently changed to:

visibility = ["//visibility:public"],

So the visibility status should be changed to `visibility = ["//visibility:public"],` also for the `schema_conversion_utils`.

Reference: https://github.com/tensorflow/tensorflow/issues/63074