Ticket #8820 (closed: fixed)
Indirect C2E: Create Info table option only works for TOSCA
Reported by: | Samuel Jackson | Owned by: | Samuel Jackson |
---|---|---|---|
Priority: | minor | Milestone: | Release 3.2 |
Component: | Indirect Inelastic | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Jay Rainey |
Description
The C2E interface has an option to output a table containing run information when running a reduction. But this does nothing for OSIRIS or IRIS.
We have two options here:
- Output a similar table for table OSIRIS/IRIS
- Remove/disable the option for OSIRIS/IRIS
Speak with Sanghamitra to confirm the appropriate action.
Change History
comment:1 Changed 7 years ago by Samuel Jackson
- Component changed from Framework to Indirect Inelastic
comment:2 Changed 7 years ago by Samuel Jackson
Spoken with TOSCA scientist. This option can just be removed as all the information is already in the sample logs.
comment:3 Changed 7 years ago by Nick Draper
- Status changed from new to assigned
Bulk move of tickets out of triage (new) to assigned at the introduction of the triage state
comment:4 Changed 7 years ago by Samuel Jackson
- Status changed from assigned to inprogress
Refs #8820 Remove option from interface.
Changeset: 9f863444b60c64f578f4e390ac504b19f4d938a2
comment:5 Changed 7 years ago by Samuel Jackson
Refs #8820 Remove redundant code from MSG reducer.
Changeset: 6edca7b42cab893c4a733a2a7901fbd1b662d283
comment:6 Changed 7 years ago by Samuel Jackson
- Status changed from inprogress to verify
- Resolution set to fixed
To Tester
I've removed the option for the Indirect C2E interface for all instruments. As mentioned above, all of the information is available in the sample logs anyway. Check that energy conversion still works and that the changes look good.
comment:7 Changed 7 years ago by Jay Rainey
- Status changed from verify to verifying
- Tester set to Jay Rainey
comment:8 Changed 7 years ago by Jay Rainey
- Status changed from verifying to closed
The Create Info Table option in the interface and all references to create_info_table in Python have been removed. The energy conversion still works.
comment:9 Changed 7 years ago by Jay Rainey
Merge remote-tracking branch 'origin/bugfix/8820_tosca_create_table_option'
Full changeset: a42e5e763fefc9812354e7f528891d06d58a1af9
comment:10 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 9661