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.

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

mdstcl's do/method on python devices closes the tree parenting the device #282

Closed
zack-vii opened this issue Jan 5, 2016 · 5 comments
Closed

Comments

@zack-vii
Copy link
Contributor

zack-vii commented Jan 5, 2016

assuming \TESTTREE::TOP:DEVICE_PATH is a device that has a method 'method(self,*arg)'
The new doMethod seems to still close all trees on return.
mdstcl is segfaulting on attempt to reopen tree.

TCL> set tree testtree
TCL> show db
000  testtree             shot: -1  [\TESTTREE::TOP]

TCL> do/method device_path method
TCL> show db

TCL> set tree testtree
Segmentation fault
@zack-vii zack-vii changed the title probably since 190/191 mdstcl's do/method on python devices closes the tree parenting the device mdstcl's do/method on python devices closes the tree parenting the device Jan 5, 2016
@zack-vii
Copy link
Contributor Author

zack-vii commented Jan 5, 2016

The issue presumed despite downgrading due to residual python packages of the most recent version.
The trees do not close unless the method opens another tree after reverting #281, #267, and #261 (TreeRef related)

@zack-vii
Copy link
Contributor Author

zack-vii commented Jan 5, 2016

The closing of the trees occurs between #281 and #267, in version 7.0.202 the method segfaults immediately

TCL> set tree testtree
TCL> do/method device_path method
Segmentation fault

@zack-vii
Copy link
Contributor Author

zack-vii commented Jan 5, 2016

never mind. in fact #281 fixed the problem but I had a hard time testing it correctly. 202 works now

@zack-vii zack-vii closed this as completed Jan 5, 2016
@MDSplusBuilder
Copy link
Contributor

Should be fixed by pull request 281 merged yesterday.

On 1/4/2016 9:02 PM, zack-vii wrote:

assuming \TESTTREE::TOP:DEVICE_PATH is a device that has a method
'method(self,*arg)'
The new doMethod seems to still close all trees on return.
mdstcl is segfaulting on attempt to reopen tree.

|TCL> set tree testtree TCL> show db 000 testtree shot: -1
[\TESTTREE::TOP] TCL> do/method device_path method TCL> show db TCL>
set tree testtree Segmentation fault |


Reply to this email directly or view it on GitHub
#282.

@tfredian
Copy link
Contributor

tfredian commented Jan 5, 2016

Sorry... had not gotten to your followup messages on this issue.

On 1/5/2016 8:00 AM, MDSplusBuilder wrote:

Should be fixed by pull request 281 merged yesterday.

On 1/4/2016 9:02 PM, zack-vii wrote:

assuming \TESTTREE::TOP:DEVICE_PATH is a device that has a method
'method(self,*arg)'
The new doMethod seems to still close all trees on return.
mdstcl is segfaulting on attempt to reopen tree.

|TCL> set tree testtree TCL> show db 000 testtree shot: -1
[\TESTTREE::TOP] TCL> do/method device_path method TCL> show db TCL>
set tree testtree Segmentation fault |


Reply to this email directly or view it on GitHub
#282.


Reply to this email directly or view it on GitHub
#282 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants