[DM-8548] Add more info in the image title in the triview - Jira

Yi Mei [X] (Inactive) created issue - 01/Dec/16 4:11 PM

Trey Roby made changes - 01/Dec/16 4:17 PM

Field	Original Value	New Value
Description	Currently the image title in the triview has filter name, like u, g, i, ... It would be nice to add more information about the image, say deepCoaddId or scienceCcdExposureId.	Currently the image title in the triview has filter name, like u, g, i, ... It would be nice to add more information about the image, say deepCoaddId or scienceCcdExposureId. Trey added: see LsstSdssRequestList.js

Trey Roby made changes - 01/Dec/16 4:17 PM

Story Points

1

Xiuqin Wu [X] (Inactive) made changes - 01/Dec/16 9:13 PM

Assignee

Cindy Wang [ cwang ]

Xiuqin Wu [ xiuqin ]

Xiuqin Wu [X] (Inactive) made changes - 06/Dec/16 5:49 PM

Assignee

Xiuqin Wu [ xiuqin ]

Lijun Zhang [ zhang ]

Lijun Zhang [X] (Inactive) made changes - 15/Dec/16 3:25 PM

Status

To Do [ 10001 ]

In Progress [ 3 ]

Hide

Permalink

Lijun Zhang [X] (Inactive) added a comment - 15/Dec/16 3:50 PM

I used the DeepCoaddID and ScienceCcdExposureID as the base and then add the filter after the base. This changes were part of the ~~DM-8157~~ which was merged.

Please verify to see if the information is enough.

Thanks!

Show

Lijun Zhang [X] (Inactive) added a comment - 15/Dec/16 3:50 PM I used the DeepCoaddID and ScienceCcdExposureID as the base and then add the filter after the base. This changes were part of the DM-8157 which was merged. Please verify to see if the information is enough. Thanks!

Lijun Zhang [X] (Inactive) made changes - 15/Dec/16 3:50 PM

Reviewers	Trey Roby [ roby ]	Yi Mei [ ymei ]
Status	In Progress [ 3 ]	In Review [ 10004 ]

Xiuqin Wu [X] (Inactive) made changes - 15/Dec/16 5:55 PM

Epic Link

~~DM-8307~~ [ 27831 ]

Xiuqin Wu [X] (Inactive) made changes - 03/Jan/17 10:46 AM

Sprint

SUIT Sprint 2016-12 [ 294 ]

SUIT Sprint 2016-12, SUIT Sprint 2017-01 [ 294, 299 ]

Xiuqin Wu [X] (Inactive) made changes - 03/Jan/17 10:46 AM

Rank

Ranked higher

Yi Mei [X] (Inactive) made changes - 03/Jan/17 4:49 PM

Summary

Add more info in the image tile in the triview

Add more info in the image title in the triview

Hide

Permalink

Yi Mei [X] (Inactive) added a comment - 04/Jan/17 10:09 AM

I pulled the dev branch (as the change is already in the dev) and found the image titles having either the deepCoaddId or the scienceExposureId plus the filter names. That's good.

The only problem is that when 5 images from the five filters are shown, their deepCoaddIds or scienceExposureIds should be different. Currently they are the same for those 5 images. Please fix it.

Show

Yi Mei [X] (Inactive) added a comment - 04/Jan/17 10:09 AM I pulled the dev branch (as the change is already in the dev) and found the image titles having either the deepCoaddId or the scienceExposureId plus the filter names. That's good. The only problem is that when 5 images from the five filters are shown, their deepCoaddIds or scienceExposureIds should be different. Currently they are the same for those 5 images. Please fix it.

Lijun Zhang [X] (Inactive) made changes - 04/Jan/17 11:34 AM

Status

In Review [ 10004 ]

In Progress [ 3 ]

Hide

Permalink

Lijun Zhang [X] (Inactive) added a comment - 04/Jan/17 4:32 PM

Can you please review this ticket? Thanks!

Show

Lijun Zhang [X] (Inactive) added a comment - 04/Jan/17 4:32 PM Can you please review this ticket? Thanks!

Lijun Zhang [X] (Inactive) made changes - 04/Jan/17 4:32 PM

Status

In Progress [ 3 ]

In Review [ 10004 ]

Lijun Zhang [X] (Inactive) made changes - 04/Jan/17 5:18 PM

Description

Currently the image title in the triview has filter name, like u, g, i, ...

It would be nice to add more information about the image, say deepCoaddId or scienceCcdExposureId.

Trey added:
see LsstSdssRequestList.js

Currently the image title in the triview has filter name, like u, g, i, ...

It would be nice to add more information about the image, say deepCoaddId or scienceCcdExposureId.

Trey added:
see LsstSdssRequestList.js

[LZ 1-4-17]
According to the discussion with Yi,
* For DeepCoadd,
         # get the baseId from the selected row:
             baseId = deepCoaddId - deepCoaddId % 8
         # for each filter, the title is
           id = baseId + filterId ( numerical addition here)
           title = id+-"+filterName
* For science CCD, its id structure is different than deepCoaddId
    scienceCcdExposureId = run+filterId + field ( string concatenation)

Lijun Zhang [X] (Inactive) made changes - 04/Jan/17 5:18 PM

Description

Currently the image title in the triview has filter name, like u, g, i, ...

It would be nice to add more information about the image, say deepCoaddId or scienceCcdExposureId.

Trey added:
see LsstSdssRequestList.js

[LZ 1-4-17]
According to the discussion with Yi,
* For DeepCoadd,
         # get the baseId from the selected row:
             baseId = deepCoaddId - deepCoaddId % 8
         # for each filter, the title is
           id = baseId + filterId ( numerical addition here)
           title = id+-"+filterName
* For science CCD, its id structure is different than deepCoaddId
    scienceCcdExposureId = run+filterId + field ( string concatenation)

Currently the image title in the triview has filter name, like u, g, i, ...

It would be nice to add more information about the image, say deepCoaddId or scienceCcdExposureId.

Trey added:
see LsstSdssRequestList.js

[LZ 1-4-17]
* For DeepCoadd,
         # get the baseId from the selected row:
             baseId = deepCoaddId - deepCoaddId % 8
         # for each filter, the title is
           id = baseId + filterId ( numerical addition here)
           title = id+-"+filterName
* For science CCD, its id structure is different than deepCoaddId
    scienceCcdExposureId = run+filterId + field ( string concatenation)

Lijun Zhang [X] (Inactive) made changes - 04/Jan/17 5:19 PM

Description

Currently the image title in the triview has filter name, like u, g, i, ...

It would be nice to add more information about the image, say deepCoaddId or scienceCcdExposureId.

Trey added:
see LsstSdssRequestList.js

[LZ 1-4-17]
* For DeepCoadd,
         # get the baseId from the selected row:
             baseId = deepCoaddId - deepCoaddId % 8
         # for each filter, the title is
           id = baseId + filterId ( numerical addition here)
           title = id+-"+filterName
* For science CCD, its id structure is different than deepCoaddId
    scienceCcdExposureId = run+filterId + field ( string concatenation)

Currently the image title in the triview has filter name, like u, g, i, ...

It would be nice to add more information about the image, say deepCoaddId or scienceCcdExposureId.

Trey added:
see LsstSdssRequestList.js

[LZ 1-4-17]
# For DeepCoadd,
         # get the baseId from the selected row:
             baseId = deepCoaddId - deepCoaddId % 8
         # for each filter, the title is
           id = baseId + filterId ( numerical addition here)
           title = id+-"+filterName
# For science CCD, its id structure is different than deepCoaddId
    scienceCcdExposureId = run+filterId + field ( string concatenation)

Lijun Zhang [X] (Inactive) made changes - 04/Jan/17 5:20 PM

Description

Currently the image title in the triview has filter name, like u, g, i, ...

It would be nice to add more information about the image, say deepCoaddId or scienceCcdExposureId.

Trey added:
see LsstSdssRequestList.js

[LZ 1-4-17]
# For DeepCoadd,
         # get the baseId from the selected row:
             baseId = deepCoaddId - deepCoaddId % 8
         # for each filter, the title is
           id = baseId + filterId ( numerical addition here)
           title = id+-"+filterName
# For science CCD, its id structure is different than deepCoaddId
    scienceCcdExposureId = run+filterId + field ( string concatenation)

Currently the image title in the triview has filter name, like u, g, i, ...

It would be nice to add more information about the image, say deepCoaddId or scienceCcdExposureId.

Trey added:
see LsstSdssRequestList.js

[LZ 1-4-2017]
# For DeepCoadd,
         Get the baseId from the selected row:
               baseId = deepCoaddId - deepCoaddId % 8
         For each filter, the title is:
              id = baseId + filterId ( numerical addition here)
              title = id+"-"+filterName
# For science CCD, its id structure is different than deepCoaddId
    scienceCcdExposureId = run+filterId + field ( string concatenation)

Hide

Permalink

Yi Mei [X] (Inactive) added a comment - 05/Jan/17 10:23 AM

Reviewed the brand origin/~~DM-8548~~-addMoreInfo:

The bug in the previous version has been fixed: Five images from the five filters have their own uniq and correct ID in the titile.

The ticket can be closed.

Show

Yi Mei [X] (Inactive) added a comment - 05/Jan/17 10:23 AM Reviewed the brand origin/ DM-8548 -addMoreInfo: The bug in the previous version has been fixed: Five images from the five filters have their own uniq and correct ID in the titile. The ticket can be closed.

Hide

Permalink

Lijun Zhang [X] (Inactive) added a comment - 05/Jan/17 11:17 AM

Reviewed by Yi.

Show

Lijun Zhang [X] (Inactive) added a comment - 05/Jan/17 11:17 AM Reviewed by Yi.

Lijun Zhang [X] (Inactive) made changes - 05/Jan/17 11:17 AM

Status

In Review [ 10004 ]

Reviewed [ 10101 ]

Hide

Permalink

Lijun Zhang [X] (Inactive) added a comment - 05/Jan/17 11:17 AM

Merged.

Show

Lijun Zhang [X] (Inactive) added a comment - 05/Jan/17 11:17 AM Merged.

Lijun Zhang [X] (Inactive) made changes - 05/Jan/17 11:17 AM

Resolution		Done [ 10000 ]
Status	Reviewed [ 10101 ]	Done [ 10002 ]

Data Management

Add more info in the image title in the triview

Details

Description

Attachments

Activity

People

Dates

Jenkins