whizard is hosted by Hepforge, IPPP Durham

Opened 9 years ago

Closed 9 years ago

#707 closed task (fixed)

Refactor interactions and evaluators module

Reported by: kilian Owned by: kilian
Priority: P5 Milestone: v2.3.0
Component: core Version: 2.2.4
Severity: critical Keywords:
Cc:

Description

... continuing the task of #706.

I've yet to decide whether state_matrix_t should become the base type for interactions and then evaluators, or state_matrix_t should be abstracted as a component. Note that there is also polarization_t.

Change History (6)

comment:1 Changed 9 years ago by Juergen Reuter

Priority: P3P0
Severity: normalcritical

This is a hilariously big task, that clashes with all other work on the trunk. We should get this task done as soon as possible, or never change some of the old-fashioned data structures. Three of my attempts were not successful, so WK definitely has to take over here.

comment:2 Changed 9 years ago by Juergen Reuter

Milestone: v2.2.6v2.3.0

This is a major blocker. It affects most of the code in many sections. All my attempts failed, so this will be moved.

comment:3 Changed 9 years ago by Juergen Reuter

According to the discussions in Würzburg, only evaluator_t should inherit from interaction_t, but not interaction_t from state_matrix_t in order to allow for different implementations of state_matrix_t.

comment:4 Changed 9 years ago by Juergen Reuter

Priority: P0P5

Some transferral from global routines to TBPs happened, but caused paramount trouble with the gfortran 4.9 and 5.0. Most probably this is not gonna happen at all.

comment:5 Changed 9 years ago by Juergen Reuter

As of r6795, evaluator_t has inherited from interaction_t, and all initializers of evaluator_t have been OO (there are no different routines distinguishing between interaction and evaluator types as inputs.). Almost all routines of interaction_t and evaluator_t have become TBPs.

comment:6 Changed 9 years ago by Juergen Reuter

Resolution: fixed
Status: newclosed

This has been done in r6798. There are still some places left for polishing interactions.f90 but this can be done once in a while. Closing.

Note: See TracTickets for help on using tickets.