Print

Print


  Branch: refs/heads/master
  Home:   https://github.com/Starlink/starjava
  Commit: 6c4a3beed49a4d81fe98adb4e0c9e4eb85966204
      https://github.com/Starlink/starjava/commit/6c4a3beed49a4d81fe98adb4e0c9e4eb85966204
  Author: Mark Taylor <[log in to unmask]>
  Date:   2015-02-26 (Thu, 26 Feb 2015)

  Changed paths:
    M topcat/src/main/uk/ac/starlink/topcat/plot2/SkyPlotWindow.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/geom/SkySys.java

  Log Message:
  -----------
  ttools: remove duplicated code


  Commit: 000359fe8156c4cd02586f0e91d1b9428305cd85
      https://github.com/Starlink/starjava/commit/000359fe8156c4cd02586f0e91d1b9428305cd85
  Author: Mark Taylor <[log in to unmask]>
  Date:   2015-02-26 (Thu, 26 Feb 2015)

  Changed paths:
    M topcat/src/main/uk/ac/starlink/topcat/plot2/BasicCoordLayerControl.java
    A topcat/src/main/uk/ac/starlink/topcat/plot2/ConfigStyler.java
    M topcat/src/main/uk/ac/starlink/topcat/plot2/FormControl.java
    M topcat/src/main/uk/ac/starlink/topcat/plot2/FormLayerControl.java
    M topcat/src/main/uk/ac/starlink/topcat/plot2/FormStylePanel.java
    M topcat/src/main/uk/ac/starlink/topcat/plot2/PlotPositionSpecifier.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/SurfaceFactory.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/config/BooleanConfigKey.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/config/ChoiceConfigKey.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/config/ConfigException.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/config/DoubleConfigKey.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/config/OptionConfigKey.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/config/SkySysConfigKey.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/config/SubrangeConfigKey.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/config/TimeConfigKey.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/geom/CubeSurfaceFactory.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/geom/PlaneSurfaceFactory.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/geom/SkySurfaceFactory.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/layer/BinSizer.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/layer/FunctionPlotter.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/layer/LabelPlotter.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/layer/ShapePlotter.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/task/AbstractPlot2Task.java

  Log Message:
  -----------
  ttools: ConfigException is now checked, not unchecked

Reparent ConfigException to inherit from Exception not RuntimeException.
It's really necessary to have it as a checked exception to make it
clear when you have to handle it.  It was only unchecked before because
I was in a tearing hurry when I put did it in the first place.

Added some code in topcat to behave sensibly when such an exception
is caught, but this is not much tested so far; currently these are
not often thrown.


  Commit: 25ae21c0e5fd0dc1f0f890d76671dfdd72929d07
      https://github.com/Starlink/starjava/commit/25ae21c0e5fd0dc1f0f890d76671dfdd72929d07
  Author: Mark Taylor <[log in to unmask]>
  Date:   2015-02-26 (Thu, 26 Feb 2015)

  Changed paths:
    M topcat/src/main/uk/ac/starlink/topcat/plot2/ConfigSpecifier.java

  Log Message:
  -----------
  topcat: provide ConfigMap check hook in ConfigSpecifier

ConfigSpecifier subclasses can now override the new checkConfig
method and throw a ConfigException if something is wrong with
the map.  The class will then do something about it to ensure
that the eventual returned config maps are safe to use.


  Commit: 8977b52fe72cd30768e8de386ef62acb6721821d
      https://github.com/Starlink/starjava/commit/8977b52fe72cd30768e8de386ef62acb6721821d
  Author: Mark Taylor <[log in to unmask]>
  Date:   2015-02-26 (Thu, 26 Feb 2015)

  Changed paths:
    M topcat/src/docs/sun253.xml
    M topcat/src/main/uk/ac/starlink/topcat/plot2/ConfigSpecifier.java
    M topcat/src/main/uk/ac/starlink/topcat/plot2/CubeAxisController.java
    M topcat/src/main/uk/ac/starlink/topcat/plot2/HistogramAxisController.java
    M topcat/src/main/uk/ac/starlink/topcat/plot2/PlaneAxisController.java
    M topcat/src/main/uk/ac/starlink/topcat/plot2/ShaderControl.java
    M topcat/src/main/uk/ac/starlink/topcat/plot2/TimeAxisController.java
    M ttools/src/main/uk/ac/starlink/ttools/plot2/PlotUtil.java

  Log Message:
  -----------
  topcat: check and fix axis ranges before it's too late

Axis range values manually entered by users in Min/Max fields
are now checked in a consistent way before use.

Previously if the user entered values with the wrong sense
(min>=max) it could lead to uncaught exceptions for the
geometric axes, though the Aux axis checked these and dealt
with them using custom code.


Compare: https://github.com/Starlink/starjava/compare/e9a6ff56fbcd...8977b52fe72c