The current baselined version of LSE-163 says:

including the full light curves.

the new version now says:

including 12 months of light curves of the SSObject or DIAObject, including all other DIASources linked with the SSObject or DIAObject.

The pull request on this ticket replaces "the full" with "12 months", the rest of the new text was added by Kian-Tat Lim a while a go. I'm not sure what the text after the comma is saying now. If we say "12 months of light curve for the DIAObject" doesn't that imply we are including the DIASources associated with that DIAObject?

Is this change enough for you?

I think it would be clearer to say,

"including all DIASources from the last 12 months that are linked with the SSObject or DIAObject."

Section 4.5.1 ("Information Contained in Each Alert") needs updating as well; it still says "All previous DIASource records."

Thanks. I've updated the pull request and updated the PDF at https://lse-163.lsst.io/v/DM-10130/dpdd.pdf

I've included your suggested wording. I've also updated section 3.5.1 as requested. I believe the bullet point referring to matching Object IDs also needed updating with the 12 months. Can you please take another look?

Eric Bellm can you take another look at this?

Looks good to me.

Tim Jenness It looks like we made this change in LSE-163 but forgot to make the corresponding change in LSE-61 to DMS-REQ-0274.

It looks does not look like this update to LSE-61 requirement  DMS-REQ-0274 was made before closing this ticket. I will create a new ticket to make the update to LSE-61