Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Keep record of patching process log files #447

Open
araichoor opened this issue Oct 31, 2022 · 2 comments
Open

Keep record of patching process log files #447

araichoor opened this issue Oct 31, 2022 · 2 comments

Comments

@araichoor
Copy link
Contributor

We did so far two rounds of patching of the fiberassign files:

We want to keep record of those files.
One suggestion was to create some svn folder in $DESI_TARGET/fiberassign/tiles/, provided that those are not too large for that.
@sbailey : let s iterate in this ticket to make the final decision.

@moustakas
Copy link
Member

If I may interject: let's try to write this "record" in the form of documentation that we can use as-is in the EDR and DR1. It's not quite a "known issue" but something like the list that @geordie666 has been curating for the imaging data releases?
https://www.legacysurvey.org/dr9/issues/

@araichoor
Copy link
Contributor Author

thanks for the suggestion.
I m happy to take the lead on this (with help on technical details on the first round of patching from @dstndstn).
I just need to know where that should be done.

one question, though: what should be mentioned?
would something like the following be in the good direction:

  • edr: the fuji redux has been launched on Jan. 24, 2022 ([desi-data 5825]), so after the 1st patching (but way before the 2nd one)
    • so the "record" could be: "there are some inconsistencies in xxx columns due to some fiberassign bug, propagated downstream" (and add some details)
  • dr1: that iron redux will use the twice-patched files, so hopefully the correct thing
    • so the "record" could be: "some differences in xxx columns could be seen w.r.t. edr; the dr1 values are the correct ones"

also, kind of related: we ll likely create a svn tag for each release (at least fuji and iron), "freezing" the fiberassign files version that went into the production.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants