search

desihub / timedomain

Time-domain code.
MIT License
3 stars 14 forks source link

desidiff Module import fix #48

Closed davidlmoutard closed 3 years ago

davidlmoutard commented 3 years ago

changed the automatic file import- previously called module directory using '..', but the timedomain directory which houses desidiff is 2 directories above, so I changed it to '../../'

AlexGKim commented 3 years ago

Why are there changes around line 103?

davidlmoutard commented 3 years ago

Why are there changes around line 103?

I believe that's where you said to change it since tid is an array- I put in your suggested for loop to iterate over all possible IDs in array.

AlexGKim commented 3 years ago

OK, you should then be explaining this in the pull request.