Ticket #9597 (closed: wontfix)

Opened 6 years ago

Last modified 5 years ago

CreateSampleWorkspace seg faults with no BinWidth parameter for EventWorkspaces

Reported by: Michael Reuter Owned by: Michael Reuter
Priority: major Milestone: Release 3.2
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Peter Peterson

Description

An event workspace does not have to be given any sort of X representation (binning) when it is created, so the algorithm shouldn't seg fault when trying to do so.

Change History

comment:1 Changed 6 years ago by Nick Draper

  • Owner changed from Nick Draper to Michael Reuter
  • Status changed from new to infoneeded

Can you include the python command you are trying to use?

I have several tests using

ws=CreateSampleWorkspace("Event")

and they seem to be fine.

I do have a known issue that xUnit is ignored whatever you set it to and xMin and XMax cannot be less than 0.

The way the algorithm work some binning must be defined for the Event data. The Distribution of the events to approximate the form of the data would not work without it.

comment:2 Changed 6 years ago by Nick Draper

There is of course no reason why you could not call rebin straight after creating it if you wanted to simulate a "virgin" event workspace. eg.

ws=CreateSampleWorkspace("Event",XMin=10000,XMax=30000,BinWidth=200) #rebin to a single bin like a raw event workspace ws=rebin(ws,Params=[10000,20000,30000],PreserveEvents=True)

comment:3 Changed 6 years ago by Nick Draper

Also there is a fix around single bin width creation in #9568 that might fix it.

comment:4 Changed 6 years ago by Nick Draper

  • Milestone changed from Release 3.2 to Backlog

Moved to Backlog at the code freeze of release 3.2

comment:5 Changed 6 years ago by Michael Reuter

  • Status changed from infoneeded to verify
  • Resolution set to wontfix

I tried this again and I can't reproduce the error. I can make an event workspace with one bin by making the bin width the size of the range between min and max.

comment:6 Changed 6 years ago by Michael Reuter

  • Milestone changed from Backlog to Release 3.2

comment:7 Changed 6 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:8 Changed 6 years ago by Peter Peterson

  • Status changed from verifying to closed

Works for me too.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10440

Note: See TracTickets for help on using tickets.