Skip to content
Snippets Groups Projects
  1. Jan 18, 2017
  2. Jan 11, 2017
  3. Dec 10, 2016
    • Edward Fauchon-Jones's avatar
      Remove anti-pattern of classmethods · 4c054494
      Edward Fauchon-Jones authored
      The `Spec` class family were originally set up to work using class
      methods. This was natural because each `Spec` derivative only
      represented a single field.
      
      With the addition of the `Amplm` and `Phaselm` specs, this was no longer
      true, with each of these specs required to represent a collection of
      fields. While initially attempted to use instances for these `Specs` and
      classes+classmethods for the remaining specs, to much off the logic
      required casing on the `Spec` being an instance of class, it was decided
      to only use classes and classmethods.
      
      In the end this resulting in a massive anti-pattern, defining a class
      factory in place of instance initialisers in the `Amplm` and `Phaselm`
      classes, and a general bad logic. It made sense to use classmethods
      before, but in no longer did.
      
      This commit refactors most classmethods as instance methods (with a
      couple left as classmethods because theydo not reference the parent
      object, and could infact be staic methods). While a couple of
      expressions required additional `()` to instance `Specs`, overall the
      logic has been greatly simplified and actually demonstrated that the
      classmethod paradigm previously used had significantly little benifit.
      4c054494
    • Edward Fauchon-Jones's avatar
      Add strain modes (specs, creation, lvcnrcheck) · 41e31138
      Edward Fauchon-Jones authored
      - Specs have been added that represent phase and amplitude modes.
      - 5 degrees with everyone order of phase and amp modes fields are now
        created by `createValidSim`
      - `lvcnrcheck` looks for and validates phase and amp modes.
      41e31138
  4. Dec 09, 2016
    • Edward Fauchon-Jones's avatar
      Refactor errors to dedicated module · 4b12b9da
      Edward Fauchon-Jones authored
      `Specs.valid` now return `Error` objects and the logic of generating
      errors and associated messages have been refactor to their own dedicated
      module.
      
      Much error message logic has now been removed from `lvcnrcheck` with
      only specific cases handled as nessesary, right now only a `Missing`
      `Error` needs to be cased.
      4b12b9da
  5. Dec 05, 2016
  6. Dec 04, 2016
  7. Nov 21, 2016
  8. Nov 14, 2016
  9. Nov 07, 2016
  10. Nov 04, 2016
Loading