Cleanup external data path validation

[adrianlizarraga]

Description

• Updates the ValidateExternalData() function:
  • Resolves symlinks when validating external data path for models loaded from memory.
    • Previously only did a lexical check that did not resolve symlinks. Now, we resolve symlinks.
  • Now requires external data path to exist.
  • Replace the (base_dir, model_dir) function parameters with just model_dir. base_dir was always derived from model_dir.
  • Skip validation for WASM builds that do not have a filesystem.
  • Return Status instead of throwing exceptions when std::filesystem functions fail.
• Updates Graph::ConvertInitializersIntoOrtValues():
  • Prevents unnecessary calls to ValidateExternalData() for external data paths that have already been validated.

Motivation and Context

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[yuslepukhin]

I am having a hard time reviewing since my tools are not working.

See the tests that I introduced in my DRAFT PR in connection with my comment.

[yuslepukhin]

This is growing to be complicated. We will need a comprehensive documentation for this function detailing all what and why.