desihub / tutorials

DESI tutorials
BSD 3-Clause "New" or "Revised" License
43 stars 16 forks source link

Problems with 19.9 release #31

Closed weaverba137 closed 4 years ago

weaverba137 commented 4 years ago

Really only one serious problem, and a few questions.

Other notes:

Exception reported in FiberAssign.ipynb

multiprocessing.pool.RemoteTraceback: 
"""
Traceback (most recent call last):
  File "/global/common/software/desi/cori/desiconda/20190804-1.3.0-spec/conda/lib/python3.6/multiprocessing/pool.py", line 119, in worker
    result = (True, func(*args, **kwds))
  File "/global/common/software/desi/cori/desiconda/20190804-1.3.0-spec/conda/lib/python3.6/multiprocessing/pool.py", line 44, in mapstar
    return list(map(*args))
  File "/global/common/software/desi/cori/desiconda/20190804-1.3.0-spec/code/fiberassign/1.2.0/lib/python3.6/site-packages/fiberassign-1.2.0-py3.6-linux-x86_64.egg/fiberassign/assign.py", line 1131, in merge_results_tile
    skymon[field] = outdata[field][sky_rows]
ValueError: no field of name BRICKID
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/global/common/software/desi/cori/desiconda/20190804-1.3.0-spec/code/fiberassign/1.2.0/bin/fiberassign", line 4, in <module>
    __import__('pkg_resources').run_script('fiberassign==1.2.0', 'fiberassign')
  File "/global/common/software/desi/cori/desiconda/20190804-1.3.0-spec/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 666, in run_script
    self.require(requires)[0].run_script(script_name, ns)
  File "/global/common/software/desi/cori/desiconda/20190804-1.3.0-spec/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 1453, in run_script
    exec(code, namespace, namespace)
  File "/global/common/software/desi/cori/desiconda/20190804-1.3.0-spec/code/fiberassign/1.2.0/lib/python3.6/site-packages/fiberassign-1.2.0-py3.6-linux-x86_64.egg/EGG-INFO/scripts/fiberassign", line 168, in <module>
    run_merge(merge_args)
  File "/global/common/software/desi/cori/desiconda/20190804-1.3.0-spec/code/fiberassign/1.2.0/lib/python3.6/site-packages/fiberassign-1.2.0-py3.6-linux-x86_64.egg/fiberassign/scripts/merge.py", line 153, in run_merge
    copy_fba=(not args.skip_raw))
  File "/global/common/software/desi/cori/desiconda/20190804-1.3.0-spec/code/fiberassign/1.2.0/lib/python3.6/site-packages/fiberassign-1.2.0-py3.6-linux-x86_64.egg/fiberassign/assign.py", line 1341, in merge_results
    results = pool.map(merge_tile, tile_map_list)
  File "/global/common/software/desi/cori/desiconda/20190804-1.3.0-spec/conda/lib/python3.6/multiprocessing/pool.py", line 266, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
  File "/global/common/software/desi/cori/desiconda/20190804-1.3.0-spec/conda/lib/python3.6/multiprocessing/pool.py", line 644, in get
    raise self._value
ValueError: no field of name BRICKID
weaverba137 commented 4 years ago

@sbailey, I think the fiberassign problem may simply be a change in command-line options. Can you take a look?

forero commented 4 years ago

@weaverba137 I fixed FiberAssignDECaLS.ipynb. I will check the other two FiberAssign notebooks.

weaverba137 commented 4 years ago

@sbailey, did #34 fix all the open problems in this ticket? Should we also create a 19.10 tag?

weaverba137 commented 4 years ago

@sbailey, as I mentioned on email, this is one of the issues in project 20.3 that is just a question that needs to be answered. Could you please take a look at this?

sbailey commented 4 years ago

PR #34 fixed the fiberassign problems (which I have now checked off), but did not do anything with redshift-database.ipynb or the redrock tutorials. OK with me to make a 19.10 tag as-is, close this ticket, and re-review the current state of the tutorials in prep for a 20.x release.

sbailey commented 4 years ago

Regarding redrock/RedrockBOSSDemo.ipynb : if it still works, great. If it doesn't but it is easy to fix (e.g. a change of command line options or similar), let's keep it alive. If it is broken for some complicated reason we can deprecate it.