Project

General

Profile

Bug #4970

ciss2isis failing on "unknown" target

Added by Klaus-Michael Aye about 1 year ago. Updated 8 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Christopher Combs
Category:
Applications
Target version:
Impact:
Software Version:
Test Reviewer:

Description

There are a bunch of Cassini ISS images that cannot be imported into ISIS because of missing translation values:

N1829225966_15.IMG
STDOUT: b''
STDERR: b'PROGRAMMER ERROR Unable to find a translation value for [TargetName, AEGAEON] in file [/Volumes/Data/Applications/ISIS3/data/cassini/translations/cassiniIss.trn].\n'
N1832830863_1.IMG
STDOUT: b''
STDERR: b'PROGRAMMER ERROR Unable to find a translation value for [TargetName, ANTHE] in file [/Volumes/Data/Applications/ISIS3/data/cassini/translations/cassiniIss.trn].\n'

Can this be added to 'ciss2isis' ?

Here's the FILE_NAMEs of ISS data that I have identified to fail:

N1643264167_1.IMG
N1643264273_1.IMG
N1643264379_1.IMG
N1643264504_1.IMG
N1643264914_1.IMG
N1643265020_1.IMG
N1643265126_1.IMG
N1643265251_1.IMG
N1829223369_1.IMG
N1829223475_1.IMG
N1829223581_1.IMG
N1829223706_1.IMG
N1829224111_1.IMG
N1829224166_1.IMG
N1829224221_1.IMG
N1829224276_1.IMG
N1829224331_1.IMG
N1829224386_1.IMG
N1829224671_1.IMG
N1829224726_1.IMG
N1829224788_1.IMG
N1829224850_1.IMG
N1829224905_1.IMG
N1829224967_1.IMG
N1829225029_1.IMG
N1829225084_1.IMG
N1829225146_1.IMG
N1829225208_1.IMG
N1829225263_1.IMG
N1829225325_1.IMG
N1829225387_1.IMG
N1829225442_1.IMG
N1829225504_1.IMG
N1829225566_1.IMG
N1829225621_1.IMG
N1829225676_1.IMG
N1829225733_1.IMG
N1829225788_1.IMG
N1829225850_1.IMG
N1829225911_1.IMG
N1829225966_15.IMG
N1832830863_1.IMG
N1832830969_1.IMG
N1832831075_1.IMG
N1832831200_1.IMG


Related issues

Related to ISIS - Question #5301: Some Cassini ISS targets can be ingested but not spiced Acknowledged

History

#1 Updated by Tammy Becker about 1 year ago

  • Category set to Applications
  • Status changed from New to Acknowledged

#2 Updated by Tammy Becker about 1 year ago

Internally, the first image listed can be found:
/pds_san/PDS_Archive/Cassini/ISS/coiss_2060/data/1643188892_1643282464/N1643264167_1.LBL, IMG (TARGET_NAME = "AEGAEON")
/pds_san/PDS_Archive/Cassini/ISS/coiss_2100/data/1829042482_1829225208/N1829223369_1.LBL, IMG (TARGET_NAME = "AEGAEON")
/pds_san/PDS_Archive/Cassini/ISS/coiss_2102/data/1832736346_1832863717/N1832830863_1.LBL, IMG (TARGET_NAME = "ANTHE")

#3 Updated by Makayla Shepherd 9 months ago

  • Assignee set to Christopher Combs

#4 Updated by Makayla Shepherd 9 months ago

  • Assignee deleted (Christopher Combs)

#5 Updated by Christopher Combs 9 months ago

  • Assignee set to Christopher Combs

#6 Updated by Makayla Shepherd 9 months ago

  • Assignee deleted (Christopher Combs)

#7 Updated by Stuart Sides 9 months ago

  • Target version set to 3.5.2 (2017-01-31 Jan)

#8 Updated by Christopher Combs 9 months ago

  • Assignee set to Christopher Combs

#9 Updated by Christopher Combs 9 months ago

  • Status changed from Acknowledged to In Progress

#10 Updated by Christopher Combs 9 months ago

  • Test Reviewer set to Stuart Sides

I have searched the pds archives for Cassini ISS labels and made a list of targets to add to the ciss2isis translation file:

ANTHE
AEGAEON
ALBIORIX
BEBHIONN
BERGELMIR
BESTLA
EARTH
ERRIAPO
ERRIAPUS
FORNJOT
GREIP
HATI
HYROKKIN
IJIRAQ
JARNSAXA
K07S4
KARI
KIVIUQ
LOGE
MUNDILFARI
NARVI
PAALIAQ
S8_2004 *
S12_2004 *
S13_2004 *
S14_2004 *
S18_2004 *
SIARNAQ
SKADI
SKATHI
SKOLL
SURTUR
SUTTUNG
SUTTUNGR
TARVOS
TARQEQ
THRYM
THRYMR
YMIR

*Starred target names will be translated to "Sky" when creating an ISIS cube

This is built on Fedora25 and available for testing at /work/projects/isis/latest/m04970

The version of ciss2isis found here is using a local translation file: src/cassini/apps/ciss2isis/cassiniIss.trn

#11 Updated by Christopher Combs 9 months ago

  • Status changed from In Progress to Resolved

#12 Updated by Tammy Becker 8 months ago

can this be built to test from astrovm4 please? Thanks.

ciss2isis from=/pds_san/PDS_Archive/Cassini/ISS/coiss_2060/data/1643188892_1643282464/N1643264167_1.LBL to=N1643264167_1.cub
ciss2isis: error while loading shared libraries: libicui18n.so.57: cannot open shared object file: No such file or directory

#13 Updated by Tammy Becker 8 months ago

  • Status changed from Resolved to Feedback

#14 Updated by Christopher Combs 8 months ago

  • Status changed from Feedback to Resolved

The library issue has been resolved. Testing now works on astrovm4.

#15 Updated by Tammy Becker 8 months ago

  • Status changed from Resolved to Feedback

1) cisstoisis worked for
N1643264167_1.cub: TargetName = Aegaeon
N1829223369_1.cub: TargetName = Aegaeon
N1832830863_1.cub: TargetName = Anthe

2) spiceinit is not available under this build, so the failures in spiceinit are under isis3nightly (/usgs/pkgs/isis3nightly2018-01-10/isis). Is this expected??

spiceinit failed:
spiceinit from= from=N1643264167_1.cub

Group = Kernels
NaifFrameCode = -82360
LeapSecond = $base/kernels/lsk/naif0012.tls
TargetAttitudeShape = ($base/kernels/pck/pck00009.tpc,
$cassini/kernels/pck/cpck15Dec2017.tpc)
TargetPosition = $base/kernels/spk/de405.bsp
InstrumentPointing = ($cassini/kernels/ck/10024_10029ra.bc,
$cassini/kernels/fk/cas_v40.tf)
Instrument = Null
SpacecraftClock = $cassini/kernels/sclk/cas00171.tsc
InstrumentPosition = $cassini/kernels/spk/100325R_SCPSE_10021_1005-
5.bsp
InstrumentAddendum = $cassini/kernels/iak/IssNAAddendum004.ti
ShapeModel = Null
InstrumentPositionQuality = Reconstructed
InstrumentPointingQuality = Reconstructed
CameraVersion = 1
Error = "Can not find [BODY653_RADII] in text kernels"
End_Group

Group = Error
Program = spiceinit
Code = 1
Message = "Unable to initialize camera model"
File = spiceinit.cpp
Line = 240
End_Group

3) lastly, the named targets should be added to the ciss2isis application documentation for the user. Including the note regarding the target names translated to SKY.

#16 Updated by Christopher Combs 8 months ago

I have added spiceinit to this build and run into the same error when running it. Some of the smaller bodies do not have NAIF support for parameters such as their body radii.

This issue seems to be out of scope for this ticket, but a new ticket referring to the added targets and their kernel support will be created with a reference to this ticket.

#17 Updated by Christopher Combs 8 months ago

  • Status changed from Feedback to Resolved

Suggested documentation changes have been made.

They are available for viewing at: /work/projects/isis/latest/m04970/isis/src/cassini/apps/ciss2isis/ciss2isis.html

The list of targets that can be ingested but not spiced has been added to the "FROM" section
The list of targets that translate to "Sky" in the output cube has been added to the "TO" section

#18 Updated by Christopher Combs 8 months ago

  • Related to Question #5301: Some Cassini ISS targets can be ingested but not spiced added

#19 Updated by Tammy Becker 8 months ago

I have reviewed the updated documentation. I actually like the statement in the note of this ticket:

Some of the smaller bodies do not have NAIF support for parameters such as their body radii.

Could the description for FROM under NOTE be:

"While we support most Cassini ISS Targets for ingestion, some of the smaller bodies do not have NAIF support for parameters such as their body radii. Therefore, spiceinit will fail.
The current list of these target bodies is: ...

The additional note in "TO" for the Sky translation is good.

Thank you!!

#20 Updated by Christopher Combs 8 months ago

Here is what I have after your feedback:

NOTE: While we support most Cassini ISS Targets for ingestion, some of the smaller bodies do not currently have support for parameters such as their body radii. Therefore, spiceinit will fail when run on cubes created with these targets. The current list of these target bodies is :...

This version is available at: /work/projects/isis/latest/m04970/isis/src/cassini/apps/ciss2isis/ciss2isis.html

#21 Updated by Tammy Becker 8 months ago

Thank you for making the change. It looks good. I think we can consider this issue closed.

BTW, thank you for taking the time to create the html for the documentation, it really helps for review.

Good job!

#22 Updated by Tammy Becker 8 months ago

  • Status changed from Resolved to Closed

#23 Updated by Klaus-Michael Aye 8 months ago

Thanks! When will this fix be released into the ISIS distribution?

#24 Updated by Christopher Combs 8 months ago

This should be in isis3.5.2

Also available in: Atom PDF