whizard is hosted by Hepforge, IPPP Durham

Opened 11 years ago

Closed 11 years ago

#617 closed defect (fixed)

changing histogram setup causes Whizard to discard an existing event sample

Reported by: fbach Owned by: kilian
Priority: P2 Milestone: v2.2.0
Component: core Version: 2.2.0beta
Severity: normal Keywords:
Cc:

Description

Changing either the setup off an existing histogram or adding a new histogram unrelated to the actual analysis both makes Whizard discard existing events.

Attachments (1)

analysis.sin (266 bytes) - added by fbach 11 years ago.

Download all attachments as: .zip

Change History (11)

Changed 11 years ago by fbach

Attachment: analysis.sin added

comment:1 Changed 11 years ago by fbach

Also a change in the analysis makes Whizard discard events. Try

analysis = record pt (eval Pt [H])

instead of filling rap in the example.

comment:2 Changed 11 years ago by kilian

Priority: P3P2
Status: newassigned

comment:3 Changed 11 years ago by kilian

I don't get it. What's wrong?

Could you attach the .sin file which doesn't behave as expected?

comment:4 Changed 11 years ago by Juergen Reuter

There is an attachment ...

comment:5 in reply to:  4 Changed 11 years ago by kilian

Replying to jr_reuter:

There is an attachment ...

Yes, and it behaves as expected, doesn't it? What do I have to modify, exactly?

comment:6 Changed 11 years ago by Juergen Reuter

Change the two lines (one being active in the first run, the second being active in the second run):

histogram rap (-5, 5, 0.5)

and

histogram rap (-5, 5, 0.2)

Then you find in the second run:

| Events: reading from raw file 'proc.evx'
| Events: parameter mismatch, discarding old event set
| Events: closing raw file 'proc.evx'
| Events: generating new events

But it shouldn't regenerate ...

comment:7 Changed 11 years ago by kilian

It looks like the histogram is not responsible at all ... I get a parameter mismatch for a modified test even if the second .sin file is identical.

Something's random in the MD5 sum source, probably.

comment:8 Changed 11 years ago by kilian

Arghh! The MD5 sum recorded by VAMP includes the output from the integration timer!

comment:9 Changed 11 years ago by kilian

Presumably fixed in r5575. Maybe someone can double-check ... it works for me now.

comment:10 Changed 11 years ago by Juergen Reuter

Resolution: fixed
Status: assignedclosed

I rechecked this. Seems to work. If we find a counterexample, we could reopen. Closing for now.

Note: See TracTickets for help on using tickets.