whizard is hosted by Hepforge, IPPP Durham

Opened 11 years ago

Closed 11 years ago

#503 closed defect (fixed)

Resurrect pdf_builtin

Reported by: Juergen Reuter Owned by: kilian
Priority: P0 Milestone: v2.2.0
Component: core Version: 2.1.1
Severity: normal Keywords:
Cc:

Description

It's not working. Seems to be the setup of the paths, which traces down through integrations to processes.

Change History (7)

comment:1 Changed 11 years ago by Juergen Reuter

With r4551, the path is correctly set again for the installed version. To prevent the dispatch unit tests to depend on external path variables, there a dummy path "." is used. Still, the PDF set cannot be correctly set yet.

comment:2 Changed 11 years ago by Juergen Reuter

Priority: P3P1

comment:3 Changed 11 years ago by Juergen Reuter

One more thing connected to comment1: At the moment, the unit test is copying the PDF table over from its place in share/pdf_builtin or so. But here also the proper test path should be used! This is connected to the qcd_parameters ticket (#513).

comment:4 Changed 11 years ago by Juergen Reuter

Priority: P1P0

The previous (2.1) pdf_builtin SINDARIN file is still not running through as still it is not possible to set other PDFs. But this might be because they are set as optional arguments? (cf. #516). Ranking it up anyhow....

comment:5 Changed 11 years ago by kilian

Resolution: fixed
Status: newclosed

Done in r4651, including alpha_s access. The PDF set can be defined globally. Local options are indeed not implemented yet, but this is a different ticket (see above). Closing this one.

comment:6 Changed 11 years ago by Juergen Reuter

Resolution: fixed
Status: closedreopened

This seems not to be correct in tests/Makefile.am, WHIZARD should know the correct paths (now):

PDF_FILE = cteq6l.tbl
$(PDF_FILE): $(top_builddir)/share/pdf_builtin/$(PDF_FILE)
        cp $< $@

sf_pdf_builtin.run: $(PDF_FILE)

So I guess this has to be removed. Right?

comment:7 Changed 11 years ago by kilian

Resolution: fixed
Status: reopenedclosed

Yes, this was obsolete. Removed in r4668.

Note: See TracTickets for help on using tickets.