Skip to content

Commit

Permalink
bug fix in test_io and new jobdesc to CUSTOMZTILE
Browse files Browse the repository at this point in the history
  • Loading branch information
akremin committed Aug 5, 2024
1 parent 96922c7 commit cb415f1
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions py/desispec/test/test_io.py
Original file line number Diff line number Diff line change
Expand Up @@ -937,12 +937,12 @@ def test_findfile(self):
tileid = 8888
refpath = os.path.expandvars(f'$DESI_SPECTRO_DATA/{night}/{expid:08d}/fiberassign-{tileid:06d}.fits.gz')
for groupname in ('healpix', 'pernight', 'cumulative', 'perexp', 'blatfoo'):
testpath = findfile('fiberassign', night=night, expid=expid, tile=tileid, groupname='healpix')
testpath = findfile('fiberassign', night=night, expid=expid, tile=tileid, groupname=groupname)
self.assertEqual(testpath, refpath)

refpath = os.path.expandvars(f'$DESI_SPECTRO_REDUX/$SPECPROD/preproc/{night}/{expid:08d}/tilepix-{tileid}.json')
for groupname in ('healpix', 'pernight', 'cumulative', 'perexp', 'blatfoo'):
testpath = findfile('tilepix', night=night, expid=expid, tile=tileid, groupname='healpix')
testpath = findfile('tilepix', night=night, expid=expid, tile=tileid, groupname=groupname)
self.assertEqual(testpath, refpath)

#- Can't set both tile and healpix
Expand Down
2 changes: 1 addition & 1 deletion py/desispec/workflow/desi_proc_funcs.py
Original file line number Diff line number Diff line change
Expand Up @@ -455,7 +455,7 @@ def determine_resources(ncameras, jobdesc, nexps=1, forced_runtime=None, queue=N
elif jobdesc == 'NIGHTLYBIAS':
ncores, runtime = 15, 5
nodes = 2
elif jobdesc in ['PEREXP', 'PERNIGHT', 'CUMULATIVE', 'CUSTOM_TILE']:
elif jobdesc in ['PEREXP', 'PERNIGHT', 'CUMULATIVE', 'CUSTOMZTILE']:
if system_name.startswith('perlmutter'):
nodes, runtime = 1, 50 #- timefactor will bring time back down
else:
Expand Down
2 changes: 1 addition & 1 deletion py/desispec/workflow/redshifts.py
Original file line number Diff line number Diff line change
Expand Up @@ -239,7 +239,7 @@ def create_desi_zproc_batch_script(group,
## Derive job description name from group
jobdesc = group
if jobdesc not in ('perexp', 'pernight', 'cumulative'):
jobdesc = 'custom_tile'
jobdesc = 'customztile'
log.warning(f'Unrecognized {group=}, using {jobdesc=}')

## If system name isn't specified, guess it
Expand Down

0 comments on commit cb415f1

Please sign in to comment.