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

TRestRawSignalRecoverChannelsProcess: removed #ifdef REST_DetectorLib… #67

Closed
wants to merge 1 commit into from

Conversation

KonradAltenmueller
Copy link
Member

@KonradAltenmueller KonradAltenmueller commented Jun 7, 2022

KonradAltenmueller Ok: 12

I removed #ifdef REST_DetectorLib from the TRestRawSignalRecoverChannelsProcess, because it else wouldn't work (resulting in a crash when running the rml).

This would quickly solve #2

@jgalan
Copy link
Member

jgalan commented Jun 7, 2022

I thought this should have been solved after merging #62. What is the problem/error coming out?

@KonradAltenmueller
Copy link
Member Author

It only crashes when running the rml, and the error message doesn't tell me anything. You can try to run /home/konrad/iaxo-quickana/rawToRawSignalTest.rml, e.g. on this data
restManager --c rawToRawSignalTest.rml --f /storage/iaxo/iaxo-lab/data/R01306_Ar2Iso_Calibration_Fe55_2h_Vm_320_Vd_95_Pr_1.4_Gain_0x1_Shape_0xD_Clock_0xA-000.aqs

@jgalan
Copy link
Member

jgalan commented Jun 7, 2022

I am getting a seg.fault when trying to execute the lines you gave me.

@KonradAltenmueller
Copy link
Member Author

Ok, forget it, it seems to work. Apparently the framework in my installation was somehow behind master (even though I am sure I pulled it this morning...)

@KonradAltenmueller KonradAltenmueller deleted the RecoverChannels_update branch June 7, 2022 16:55
@jgalan
Copy link
Member

jgalan commented Jun 7, 2022

Still, there is a problem in your RML,

konrad@sultan:~/iaxo-quickana$ restManager --c rawToRawSignalTest.rml --f /storage/iaxo/iaxo-lab/data/R01306_Ar2Iso_Calibration_Fe55_2h_Vm_320_Vd_95_Pr_1.4_Gain_0x1_Shape_0xD_Clock_0xA-000.aqs
Warning in <UnknownClass::SetDisplay>: DISPLAY not set, setting it to 87.100.43.164:0.0
Starting pid: 16923
                            
              [==                                  Launching TRestManager...                                   ==]              
                            
              [==                                    Creating TRestManager                                     ==]              
              [==                                            path:                                             ==]              
Method for parsing string to TFile has not been registered!

That error message is coming from the fact that you are using an obsolete outputFile parameter inside TRestRun. It should be replaced by outputFileName.

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

Successfully merging this pull request may close these issues.

2 participants