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

Proposed changes to fs-to-dc #515

Open
SpacemanPaul opened this issue Oct 19, 2022 · 2 comments
Open

Proposed changes to fs-to-dc #515

SpacemanPaul opened this issue Oct 19, 2022 · 2 comments
Assignees

Comments

@SpacemanPaul
Copy link
Contributor

SpacemanPaul commented Oct 19, 2022

See also #367

Again we are finding use cases where we would like to be able to pass multiple file paths to fs-to-dc rather than a base directory and an explicit glob pattern.

The concerns raised by @Kirill888 in #367 have some merit, but do not apply to the specific use cases we are looking at.

I propose the following backwards-compatible behaviour:

Add a new flag option --files:

  • If not set (the default), fs-to-dc works as it does now - with an input directory and an optional --glob handled internally.
  • If set, then explicit file paths are expected. The "input directory" parameter is interpreted as the first path, and additional paths can be supplied up to the maximum command line length.
  • Setting both --files and --glob is an error.

I have reason to suspect that fs-to-dc is not heavily used. (When I updated it recently it had a serious bug that rendered it unusable.)

@SpacemanPaul SpacemanPaul self-assigned this Oct 19, 2022
@dunkgray
Copy link

Overall adding the dataset maturity concept to "datacube dataset add" would be my favoured approach. I haven't ever used odc-tools on NCI and I would prefer keeping the code dependencies down.

@omad
Copy link
Member

omad commented Oct 20, 2022

That sounds like a good compromise, thank you Paul.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants