dedup _get_model_external_data_paths

[xieofxie]

What does this PR do?

Fixes #2216

Before submitting

• This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
• Did you make sure to update the documentation with your changes?
• Did you write any new necessary tests?

Who can review?

[IlyasMoutawwakil]

can you provide a concrete case where this happens ?

[xieofxie]

when using Olive, I could get models like this
model.zip

sample code

import onnx
from onnx.external_data_helper import ExternalDataInfo, _get_initializer_tensors
from pathlib import Path
from collections import Counter

model_path = Path("./model.onnx")

# _get_model_external_data_paths 
onnx_model = onnx.load(str(model_path), load_external_data=False)
model_tensors = _get_initializer_tensors(onnx_model)
# filter out tensors that are not external data
model_tensors_ext = [
    ExternalDataInfo(tensor).location
    for tensor in model_tensors
    if tensor.HasField("data_location") and tensor.data_location == onnx.TensorProto.EXTERNAL
]
data = [model_path.parent / tensor_name for tensor_name in model_tensors_ext]

data_count = Counter(data)
print(data_count)

Got Counter({WindowsPath('./weights.pb'): 685})

[IlyasMoutawwakil]

okay I see

[HuggingFaceDocBuilderDev]

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

[IlyasMoutawwakil]

@xieofxie please run pip install .[quality] and then make style

[xieofxie]

Internal Error 。。

[IlyasMoutawwakil]

@xieofxie yeah those are okay, merging !