Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

PythonInterface casting for workspaces can't work with multiple inheritance, e.g IMaskWorkspace #9839

Closed
martyngigg opened this issue Feb 12, 2014 · 1 comment
Closed

PythonInterface casting for workspaces can't work with multiple inheritance, e.g IMaskWorkspace #9839

martyngigg opened this issue Feb 12, 2014 · 1 comment
Assignees
@martyngigg
Labels
Extra Attention Testers and Gate keepers should pay extra attention as this affects core aspects. High Priority An issue or pull request that if not addressed is severe enough to postponse a release.
Milestone
Release 3.2

Comments

@martyngigg
Copy link
Member

This ticket is blocks : TRAC8304, TRAC8943

It was found while trying to implement http://trac.mantidproject.org/mantid/ticket/8943 that the automatic downcast of DataItem pointers when returning to Python doesn't work for objects that multiple inherit from several interfaces.

The current mechanism tries to determine the required Python class automagically but fails in these types of circumstance and due to how boost python works it can't be made to work.

This ticket will be about implementing a simpler solution than the auto casting. It will simply have a C++ map of string IDs to the C++ types that they should be exposed as when returned to Python. This will also allow us more flexibility in choosing what is returned.

Keywords: CORE
@martyngigg
Copy link
Member Author

This issue was originally trac ticket 8996

@martyngigg martyngigg added High Priority An issue or pull request that if not addressed is severe enough to postponse a release. Python Extra Attention Testers and Gate keepers should pay extra attention as this affects core aspects. labels Jun 3, 2015
@martyngigg martyngigg self-assigned this Jun 3, 2015
@martyngigg martyngigg added this to the Release 3.2 milestone Jun 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@martyngigg martyngigg

Labels
Extra Attention Testers and Gate keepers should pay extra attention as this affects core aspects. High Priority An issue or pull request that if not addressed is severe enough to postponse a release.
Projects
None yet
Milestone
Release 3.2
Development

No branches or pull requests
1 participant
@martyngigg