Update surveysim to match recent desisurvey updates, particularly regarding fiber assignment. #70
Conversation
|
@dkirkby , this PR needs to be merged to get desisurvey and surveysim to work together following the updates to desisurvey. Some of the build failures look like gobbledygook to me. Others are clearly related to not having a recent enough desisurvey installed in the travis tests. I don't know how to specify a correct github commit in .travis.yml to resolve this. |
|
By tweaking some not obviously relevant lines of .travis.yml to get them to look more like the equivalent file in desisurvey, this now passes the Travis tests. I'll plan to merge it tommorow absent objections. @dkirkby , I don't have any real familiarity with travis; if you could confirm that I haven't done anything crazy I would appreciate it. Thanks! |
|
Can the branch |
|
Thanks Ben. I went through and cleared these. |
This PR merges changes to surveysim needed to accommodate changes to desisurvey regarding fiber assignment and the use of a single status file rather than planner & scheduler status files.