Skip to content

Commit

Permalink
RF: upload - hide away buggy -d, and remove -d short from devel girde…
Browse files Browse the repository at this point in the history
…r option

See #52 for demonstration of
incorrect behavior when -d is specified.  Disabling is a temporary workaround
  • Loading branch information
yarikoptic committed Mar 16, 2020
1 parent 9df99d6 commit 55f70ad
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions dandi/cli/cmd_upload.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,11 @@


@main.command()
@dandiset_path_option(
help="Top directory (local) of the dandiset. Files will be uploaded with "
"paths relative to that directory. If not specified, current or a parent "
"directory containing dandiset.yaml file will be assumed "
)
# @dandiset_path_option(
# help="Top directory (local) of the dandiset. Files will be uploaded with "
# "paths relative to that directory. If not specified, current or a parent "
# "directory containing dandiset.yaml file will be assumed "
# )
@click.option(
"-e",
"--existing",
Expand Down Expand Up @@ -60,7 +60,7 @@
"-c", "--girder-collection", help="For development: Girder collection to upload to"
)
# TODO: figure out folder for the dandiset
@devel_option("-d", "--girder-top-folder", help="For development: Girder top folder")
@devel_option("--girder-top-folder", help="For development: Girder top folder")
#
@devel_option(
"--fake-data",
Expand All @@ -72,9 +72,9 @@
@map_to_click_exceptions
def upload(
paths,
dandiset_path,
existing,
validation,
existing="refresh",
validation="require",
dandiset_path=None,
# Development options should come as kwargs
girder_collection=collection_drafts,
girder_top_folder=None,
Expand Down

0 comments on commit 55f70ad

Please sign in to comment.