…ANDI archive + publish development

…f tuple elements

It became obvious after RFing to use a helper function

Common code from girder.py was moved into download.py etc.
Found these changes uncomitted after a long period of time, committing
before progressing forward with development

…e going through individual elements

Probably overdone/ugly and should be easier/better with async. TODO REDO later

…ecksuming

…rs work

See referenced in the diff issues about "modified" etc.
Also I removed block of code to fetch metadata for an asset if missing - API
now returns asset records with metadata

… consistently

Closes gh-89

Download of released datasets is broken, independent issue:
dandi/dandi-publish#112

…path value

* origin/master:
  Test upload of file second time failing with `existing="error"`
  Delete a straggler comment
  Put more of local_docker inside `try`
  Test upload function
  Move upload code to a function
  Document Sourcegraph in DEVELOPMENT.md
  Remove DANDI_DEVEL testing setting
  Test register() function directly instead of cmd_register
  Rename local-docker-compose instance to local-docker-tests
  Test register with & without a dandiset.yaml file
  Add a comment
  Allow local-docker and local-docker-compose to exist side by side
  Split off register code into a Python function
  Skip local_docker tests on Windows
  Assert that stdout from register is empty
  Got register test to work
  Include docker-compose.yml in sdist and wheel
  Unfinished smoke test for `register`
  Use an unscoped key for local testing
  [GH-27] Define a `local_docker` fixture

…r issues which would add them

With the hope that it would make test faster and also avoid sporadic
test failures on some runs on CI where it does manage to get to the 2nd
element before exception is raised:

    >       assert got == [0]
    E       assert [0, 1] == [0]
    E         Left contains one more item: 1
    E         Full diff:
    E         - [0]
    E         + [0, 1]

* origin/master:
  ENH: test code paths for when DANDI_LOG_GIRDER is set when testing with 3.6
  TST: test that our progress reporter does not blow if tqdm blows
  register(): Adjust dandi_instance argument
  Remove uses of "known_instances" in register & upload tests
  ENH(code,TST): girder - sleep at least 1 sec between retries, test retries loop
  BF: just issue a warning and no progress indication if tqdm pbar fails to initialize
  Apply review suggestions
  BF(workaround): do not fail test workflow if codecov fails
  local_docker_compose: Get Girder address from known_instances
  Add "instance" key to local_docker_compose return value
  Include "www." in ABOUT_URL
  Add Dockerized redirector to Docker Compose
  Add fixture for setting DANDI_API_KEY from local_docker_compose
  Make local_docker_compose return instance_id

Conflicts:
	dandi/consts.py - took master version but added the last None for
                      the publish/API with an issue referenced
	dandi/girder.py - just took +1 for random.random() since we no
                      longer use that code path here
	dandi/tests/test_download.py - RFed test to test retries on
                      getDownloaderAsIterator instead for Girder

* origin/master: (24 commits)
  Remove duplicate "requests" dependency
  get_instance(): Immediately return known instances that lack redirector URLs
  Raise helpful error if /server-info returns invalid version
  Make redirector report http://localhost:8081 as Girder URL
  Test uploading an unregistered dandiset
  Assert a certain code path is unreachable
  Test that lock_dandiset() is called when uploading
  Tests of lock_dandiset()
  Remove unused monkeypatches
  Configure flake8 to run on test code as well
  Detect nonexistent Dandisets when uploading
  ENH: upload - add locking of the dandiset
  Make pytest ignore DeprecationWarnings from responses
  Fetch all commits in test workflow
  Replace some occurrences of known_instances with get_instance()
  Test get_instance()
  CliVersionTooLowError → CliVersionTooOldError
  Move CLI version exceptions to dandi/exceptions.py
  Use semantic_version instead of packaging
  [GH-184] Add a get_instance() function
  ...

 Conflicts:
	dandi/download.py
	dandi/girder.py
	dandi/tests/test_download.py - conflicted with a linting change
	dandi/tests/test_utils.py - both added tests

 No conflicts but needed changes:
    dandi/utils.py -- needed to add dandiapi into returned dandi_instance

…"api"

I left module and class still with dandiapi names

…ffect)

It did not rebase easily into merge commit so I decided to keep it as is

…download

Which also removes a typoed default value - duplication is evil!

Originally it was thought that may be resolution for
dandi/dandi-publish#78
would make metadata return optional, but it is always returned,
so no need for the option

… names

…e than expected