Mixing of file_list and file_name? about treecorr

[rmjarvis]

Sorry, I forgot to write a unit test for that, so of course that means I screwed it up. Try out this branch (#10) and see if it works for you now.

[reikonakajima]

I tried branch #10, but still getting the same error:

corr2 corr2.kids.params file_list=wtheta/KIDS_238p0_m0p5_MAG_AUTO.list rand_file_name=randmask/KIDS_238p0_m0p5_rand.fits n2_file_name=wtheta/NorthEnd_MAG_AUTO.n2out 
Traceback (most recent call last):
  File "/users/reiko/bin/corr2", line 5, in <module>
    pkg_resources.run_script('TreeCorr==3.0.2', 'corr2')
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 499, in run_script
    self.require(requires)[0].run_script(script_name, ns)
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 1235, in run_script
    execfile(script_filename, namespace, namespace)
  File "/users/reiko/lib/python2.7/site-packages/TreeCorr-3.0.2-py2.7-linux-x86_64.egg/EGG-INFO/scripts/corr2", line 105, in <module>
    main()
  File "/users/reiko/lib/python2.7/site-packages/TreeCorr-3.0.2-py2.7-linux-x86_64.egg/EGG-INFO/scripts/corr2", line 102, in main
    treecorr.corr2(config, logger)
  File "/users/reiko/lib/python2.7/site-packages/TreeCorr-3.0.2-py2.7-linux-x86_64.egg/treecorr/corr2.py", line 228, in corr2
    raise AttributeError("Either file_name or file_list is required")
AttributeError: Either file_name or file_list is required

[rmjarvis]

I don't see anything else wrong. I have tests now that explicitly have a file_list in conjunction with rand_file_name and they seem to work fine for me. My best guess is that your file with the list of other file names isn't being read correctly. So I added a warning about this that may give more information.

If that doesn't seem to be it, could you send me your exact config file and input files that are failing for you?

[reikonakajima]

Thanks for the new error message:

Warning: file_list provided, but no names were read from the file wtheta/KIDS_238p0_m0p5_MAG_AUTO.list

I was passing an empty list (sorry). Filling it out fixed the problem.

[reikonakajima]

Thanks again!

[rmjarvis]

Great. I'll merge this into master then.