Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only export forward slashes to tables contained in InVEST datastacks #1480

Open
phargogh opened this issue Dec 14, 2023 · 0 comments
Open

Only export forward slashes to tables contained in InVEST datastacks #1480

phargogh opened this issue Dec 14, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@phargogh
Copy link
Member

phargogh commented Dec 14, 2023

Suppose:

  • We are running InVEST on Windows
  • We are running a model that uses an input table T that contains a path to a file on disk
  • We want to export the model's collection of inputs to a datastack

When the above conditions are met, the current output datastack contains a copy of table T, T2 where all filepaths have been rewritten to be relative to table T2. Since we are running on Windows, these paths contain backslashes representing directory separators.

It would improve compatibility (see #1479 for a related issue) when opening the datastack on a mac/linux computer if we were to save the relative filepaths in T2 exclusively using / as a directory separator. Doing so would not affect compatibility on Windows, as Windows supports using either \ or / as path separators.

@phargogh phargogh added the enhancement New feature or request label Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant