[DM-11900] Add pybind11 wrappers for not-directly-tested code in meas_base - Jira

XML

Word

Printable

Details

Type: Story
Status: Invalid
Resolution: Done
Fix Version/s: None
Component/s: meas_base
Labels:
- SciencePipelines

Story Points:
4

Description

meas_base is apparently missing some pybind11 wrappers (CentroidResultKey in particular, though I have no doubt this is just one of several classes) because we didn't directly test those classes in Python.

We should add both wrappers and unit tests for these classes.

Attachments

Issue Links

mentioned in: Page Loading...

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Tim Jenness added a comment - 28/Jun/21 11:12 PM

CentroidResultKey does seem to be wrapped and was wrapped before this ticket was filed. I'm clearly missing something.

Show

Tim Jenness added a comment - 28/Jun/21 11:12 PM CentroidResultKey does seem to be wrapped and was wrapped before this ticket was filed. I'm clearly missing something.

Hide

Permalink

Jim Bosch added a comment - 29/Jun/21 1:41 AM

Just marked it as Invalid. It's plausible that there's something not wrapped and not tested, but if nobody has noticed its absence in Python maybe it doesn't merit wrapping.

Show

Jim Bosch added a comment - 29/Jun/21 1:41 AM Just marked it as Invalid. It's plausible that there's something not wrapped and not tested, but if nobody has noticed its absence in Python maybe it doesn't merit wrapping.

People

Assignee:: Unassigned

Reporter:: Jim Bosch

Watchers:: Jim Bosch, Tim Jenness

Votes:: 0 Vote for this issue

Watchers:: 2 Start watching this issue

Dates

Created:: 13/Sep/17 7:45 AM

Updated:: 29/Jun/21 1:41 AM

Resolved:: 29/Jun/21 1:40 AM

Jenkins

No builds found.

Data Management