Refactoring #1766

Core: FF: Replace all form factors in sample creation by shapes

Added by herck almost 4 years ago. Updated 2 months ago.

Status:BacklogStart date:16 Feb 2017
Priority:NormalDue date:
Assignee:wuttke% Done:

0%

Category:-
Target version:-

Description

A simple particle would then contain a shape and a material. Internally, the correct form factors, which still exist, would be generated during a computation.
This would accomplish:
  • Clearer separation between particles' description and the underlying computations (as is mostly the case for all other objects)
  • No need for onChange(), since the shapes would be purely defined by their constructor arguments
  • The shape classes could be shared inside the GUI, allowing an unified generation of vertices for 3D view and some calculations
  • This shared description would then also allow easier implementation of support for 3D files describing a certain shape
  • Custom form factors can then also indicate their real space sizes to determine if they cross layer interfaces

For custom form factors in Python, a specific shape could be used that accepts a (overriden) form factor. This would also be the approach in case of shapes defined by some 3D file format.

Related to #1567.


Related issues

Related to BornAgain - Refactoring #2457: Code organisation: facilitate sample model API changes Backlog 18 May 2020
Related to BornAgain - Refactoring #2466: Use full paths in #include directives Resolved 23 Jun 2020

History

#1 Updated by herck almost 4 years ago

  • Description updated (diff)

#2 Updated by herck almost 4 years ago

  • Description updated (diff)

#3 Updated by wuttke 5 months ago

  • Related to Refactoring #2457: Code organisation: facilitate sample model API changes added

#4 Updated by wuttke 5 months ago

#5 Updated by wuttke 5 months ago

  • Status changed from Backlog to Sprint
  • Assignee set to wuttke

Whether we call the form factors or shapes, is of secondary importance. The key point is to avoid duplication of classes. So I will merge classes FormFactorX and X for all X.

#6 Updated by wuttke 2 months ago

  • Subject changed from Replace all form factors in sample creation by shapes to Core: FF: Replace all form factors in sample creation by shapes
  • Status changed from Sprint to Backlog
  • Priority changed from Normal to High

#7 Updated by wuttke 2 months ago

  • Priority changed from High to Normal

Also available in: Atom PDF