unageanu-jiji 1.0.1 → 1.0.2

data/ChangeLog

@@ -1,2 +1,10 @@
-2009--3-29
+
+2009-4-14
+    * 以下の不具合を修正。
+        - システムを再起動すると、動作中のエージェントが全て停止状態となる。
+        - リアルトレードで建玉にエージェント名が設定されない場合がある。
+        - リアルトレードで追加したエージェントのグラフ設定ができない。
+        - 月の初めに、取引一覧の範囲絞込みの初期値が不正な値になる。
+
+2009-3-29
     * 初版リリース

data/html/js/result-page.js

@@ -205,8 +205,8 @@ fx.ui.pages.TradeResultPage.prototype = {
         endDate = now.getTime()/1000;
 
         // フィールドにも設定しておく
-        document.getElementById("subpage-trade_range-year").value = now.getFullYear();
-        document.getElementById("subpage-trade_range-month").value = now.getMonth()+1;
+        document.getElementById("subpage-trade_range-year").value = start.getFullYear();
+        document.getElementById("subpage-trade_range-month").value = start.getMonth()+1;
         document.getElementById("subpage-trade_range-day").value = start.getDate();
         document.getElementById("subpage-trade_range-end-year").value = now.getFullYear();
         document.getElementById("subpage-trade_range-end-month").value = now.getMonth()+1;

data/lib/jiji/process.rb

@@ -15,9 +15,10 @@ module JIJI
 
     # コンストラクタ
     # 再起動後の復元の場合、プロパティを指定しないこと。この場合設定ファイルからロードされる。
-    def initialize( id, process_dir, agent_manager, props=nil, registry=nil, ignore_error=false )
-      
+    def initialize( id, process_dir, agent_manager, logger, props=nil, registry=nil, rmt=false )
+
       is_recreate = props == nil
+      @logger = logger
       @registry = registry
       @id = id
       @agent_manager = agent_manager
@@ -43,20 +44,20 @@ module JIJI
         }
         @props["agents"] = [] unless @props.key? "agents"
       end
-      
-      # 新規作成の場合はエージェントをロード
-      # 再起動後の再作成時は、アウトプロットのみ作成。
+
+      # RMTまたは新規作成の場合はエージェントをロード
+      # 再起動後の再作成時は、アウトプットのみ作成。
       @outputs = {}
-      if !is_recreate
-        load_agent(ignore_error)
+      if rmt || !is_recreate
+        load_agent(rmt) # リアル取引の再起動時は発生したエラーを無視する
       else
         if @props && @props["agents"]
           @props["agents"].each {|v|
-            @outputs[v["id"]] = @registry.output( @id, v["id"] )
+            @outputs[v["id"]] = @registry.output( @id, v["name"] )
           }
         end
       end
-          
+
       # 取引の有効状態を更新
       @agent_manager.operator.trade_enable =
         @props["trade_enable"] ? true : false
@@ -113,7 +114,6 @@ module JIJI
     end
 
     def []=(k,v)
-      @props[k] = v
       if ( k == "agents" )
         # エージェントの設定が更新された
         # 削除対象を特定するため、登録済みエージェントのIDのセットを作成
@@ -124,13 +124,16 @@ module JIJI
           set.delete item["id"]
         }
         # Mapに含まれていないエージェントは削除
-        set.each { |id| agent_manager.remove( id ) }
+        set.each { |id|
+          agent_manager.remove( id )
+          outputs.delete( id )
+        }
       end
       if ( k == "trade_enable" )
         # 取引の有効状態を更新
-        @agent_manager.operator.trade_enable =
-          @props["trade_enable"] ? true : false
+        @agent_manager.operator.trade_enable = v ? true : false
       end
+      @props[k] = v
       save_props
     end
     def [](k)
@@ -151,8 +154,9 @@ module JIJI
     attr :process_dir, true
     attr :agent_manager, true
     attr :outputs, true
+    attr :logger, true
     attr :registry, true
-    
+
   private
 
     # 任意のエージェントの設定を更新する。
@@ -163,7 +167,8 @@ module JIJI
         a.agent.properties = props
       else
         agent = agent_manager.agent_registry.create( cl, props )
-        agent_manager.add( id, agent )
+        agent_manager.add( id, agent, props["name"] )
+        @outputs[id] = agent.output
       end
     end
 
@@ -192,10 +197,14 @@ module JIJI
 	          agent_manager.add( v["id"], agent, v["name"] )
             @outputs[v["id"]] = agent.output
           rescue Exception
-            raise $! unless ignore_error 
+            unless ignore_error
+              raise $!
+            else
+              @logger.error $!
+            end
             # リアルトレードの場合、停止中にエージェントが破棄された場合を考慮し
             # エージェントの初期化で失敗しても無視し、出力先だけ作成
-            @outputs[v["id"]] = @registry.output( @id, v["id"] )
+            @outputs[v["id"]] = @registry.output( @id, v["name"] )
           end
 	      }
       end

data/lib/jiji/registry.rb

@@ -179,7 +179,7 @@ module JIJI
 
         # RMTプロセス
         r.register( :rmt_process ) {
-          c = JIJI::Process.new("rmt", r.process_dir, r.agent_manager("rmt",true), nil, r, true)
+          c = JIJI::Process.new("rmt", r.process_dir, r.agent_manager("rmt",true), r.process_logger("rmt"), nil, r, true)
           c.observer_manager = r.rmt_observer_manager
           c.collector = r.rmt_collector
           c
@@ -187,8 +187,7 @@ module JIJI
         # バックテストプロセス
         r.register( :backtest_process, :model=>:multiton_initialize ) {|c,p,id, props|
           # 既存のバックテストを読み込む場合、プロパティはnil
-          # このときエージェントの初期化で失敗しても無視する。(テスト実行後にエージェントが書き換えられた場合に起こりえる。)
-          c = JIJI::Process.new(id, r.process_dir, r.agent_manager(id,false), props, r, props == nil)
+          c = JIJI::Process.new(id, r.process_dir, r.agent_manager(id,false), r.process_logger(id), props, r, false)
           c.observer_manager = r.backtest_observer_manager(id)
           c.collector = r.backtest_collector(id,
             Time.at( c["start_date"]), Time.at( c["end_date"]))

data/lib/jiji/server.rb

@@ -22,8 +22,7 @@ module JIJI
       
       param[:Logger] = registry[:server_logger]  
       param[:DocumentRoot] = File.expand_path( "#{__FILE__}/../../../html" )
-      
-      param[:Logger].info( File.expand_path( "#{__FILE__}/../../../html" ) )
+      param[:Logger].info( "document root: #{ param[:DocumentRoot] }"  )
       
       start_callback_org = param[:StartCallback]
       param[:StartCallback] = proc {

data/test/test_Process.rb

@@ -22,34 +22,34 @@ class ProcessTest <  RUNIT::TestCase
   def setup
     @dir = File.dirname(__FILE__) + "/ProcessTest.tmp"
     FileUtils.mkdir_p @dir
+
+    @logger = Logger.new STDOUT
+
+    # レジストリ
+    @registry_mock = Object.new
+    class << @registry_mock
+      def output( agent_name, dir )
+        return ["from_registry"]
+      end
+    end
   end
 
   def teardown
     FileUtils.rm_rf @dir
   end
 
+  # 基本動作のテスト
   def test_basic
 
     # 新規に作成
-    agent_mng = JIJI::AgentManager.new( "aaa", RegistryMock.new, Logger.new(STDOUT))
-    agent_mng.operator = Struct.new(:trade_enable).new(true)
-    agent_mng.conf = CONF
-    agent_mng.conf.set( [:agent,:safe_level], 0)
+    agent_mng = new_agent_mang
 
-    registry_mock = Object.new
-    class << registry_mock
-      def output( agent_name, dir )
-        output = JIJI::Output.new( agent_name, "#{dir}/out" )
-        return output
-      end
-    end
-    agent_mng.registry = registry_mock
-
-    p1 = JIJI::Process.new( "1", @dir, agent_mng, {} )
+    p1 = JIJI::Process.new( "1", @dir, agent_mng, @logger, {}, @registry_mock )
     assert_equals p1.id, "1"
     assert_equals p1.props, {"agents"=>[]}
     assert_equals p1["x"], nil
     assert_equals p1.agent_manager, agent_mng
+    assert_equals p1.outputs, {}
 
     p1["x"] = "xxx"
     assert_equals p1["x"], "xxx"
@@ -92,6 +92,7 @@ class ProcessTest <  RUNIT::TestCase
     assert_equals agent_mng.get("aaa").agent.properties, {"x"=>30, "y"=>40 }
     assert_equals agent_mng.get("bbb").agent.cl, "testclass2@foo.rb"
     assert_equals agent_mng.get("bbb").agent.properties, {"x"=>30, "y"=>41 }
+    assert_equals p1.outputs, { "aaa"=>["out1","out2"], "bbb"=>["out1","out2"] }
 
     # 変更
     agents = [
@@ -118,6 +119,7 @@ class ProcessTest <  RUNIT::TestCase
     assert_equals agent_mng.get("aaa").agent.properties, {"x"=>50, "y"=>40 }
     assert_equals agent_mng.get("bbb").agent.cl, "testclass2@foo.rb"
     assert_equals agent_mng.get("bbb").agent.properties, {"x"=>60, "y"=>41 }
+    assert_equals p1.outputs, { "aaa"=>["out1","out2"], "bbb"=>["out1","out2"] }
 
     # 削除
     agents = [
@@ -135,10 +137,12 @@ class ProcessTest <  RUNIT::TestCase
     assert_equals agent_mng.get("aaa").agent.cl, "testclass@foo.rb"
     assert_equals agent_mng.get("aaa").agent.properties, {"x"=>50, "y"=>41 }
     assert_equals agent_mng.get("bbb"), nil
+    assert_equals p1.outputs, { "aaa"=>["out1","out2"] }
 
-
-    # 再読み込み / ファイルに保存された設定値が読み込まれる。
-    p1 = JIJI::Process.new( "1",  @dir, agent_mng, nil, registry_mock )
+    # RMTモードで再読み込み
+    # ファイルに保存された設定値が読み込まれる。また、エージェントがインスタンス化される
+    agent_mng = new_agent_mang
+    p1 = JIJI::Process.new( "1",  @dir, agent_mng, @logger, nil, @registry_mock, true )
     assert_equals p1.id, "1"
     assert_equals p1.agent_manager, agent_mng
     assert_equals p1["x"], "xxx"
@@ -148,16 +152,168 @@ class ProcessTest <  RUNIT::TestCase
     assert_equals agent_mng.get("bbb"), nil
     assert_equals p1["trade_enable"], false
     assert_equals p1.agent_manager.operator.trade_enable, false
+    assert_equals p1.outputs, { "aaa"=>["out1","out2"] }
+
+    # バックテストモードで再読み込み
+    # ファイルに保存された設定値が読み込まれるが、エージェントはインスタンス化されない
+    agent_mng = new_agent_mang
+    p1 = JIJI::Process.new( "1",  @dir, agent_mng, @logger, nil, @registry_mock, false )
+    assert_equals p1.id, "1"
+    assert_equals p1.agent_manager, agent_mng
+    assert_equals p1["x"], "xxx"
+    assert_equals p1["agents"], agents
+    assert_equals agent_mng.get("aaa"), nil
+    assert_equals agent_mng.get("bbb"), nil
+    assert_equals p1["trade_enable"], false
+    assert_equals p1.agent_manager.operator.trade_enable, false
+    assert_equals p1.outputs, { "aaa"=> ["from_registry"] }
 
     # 別のプロセスを作成 / 設定値は別途保持される。
-    p2 = JIJI::Process.new( "2", @dir, agent_mng, {}, registry_mock )
+    agent_mng = new_agent_mang
+    p2 = JIJI::Process.new( "2", @dir, agent_mng, @logger, {}, @registry_mock )
     assert_equals p2.id, "2"
     assert_equals p2.agent_manager, agent_mng
     assert_equals p2["x"], nil
     assert_equals p2["agents"], []
+    assert_equals p2.outputs, { }
+
+    # 作成の段階でエージェントを指定する
+    agent_mng = new_agent_mang
+    p3 = JIJI::Process.new( "3", @dir, agent_mng, @logger, {"agents"=>agents}, @registry_mock )
+    assert_equals p3.id, "3"
+    assert_equals p3.agent_manager, agent_mng
+    assert_equals p3["x"], nil
+    assert_equals p3["agents"], agents
+    assert_equals agent_mng.get("aaa").agent.cl, "testclass@foo.rb"
+    assert_equals agent_mng.get("aaa").agent.properties, {"x"=>50, "y"=>41 }
+    assert_equals agent_mng.get("bbb"), nil
+    assert_equals p3.outputs, { "aaa"=>["out1","out2"] }
 
   end
 
+  # エージェントの作成でエラーになった場合のテスト
+  def test_error
+    agents = [
+      {"id"=>"aaa",
+       "name"=>"テストエージェント1",
+       "class"=>"testclass@foo.rb",
+       "class_name" => "testclass",
+       "file_name" => "foo.rb",
+       "description" => "説明",
+       "property_def" => {"id"=>"x", "default"=>85 },
+       "properties" => {"x"=>50, "y"=>41 }}
+    ]
+
+    # RMTで生成時にエラー
+    # 再起動で失敗するのを防ぐため、エラーはログに出力された後、無視される。
+    agent_mng = new_agent_mang
+    p1 = JIJI::Process.new( "1", @dir, agent_mng, @logger, {"agents"=>agents}, @registry_mock, true )
+
+    error_registry = RegistryMock.new
+    class << error_registry
+      def create(*args); raise "test"; end
+    end
+    # 再作成
+    agent_mng = new_agent_mang(error_registry)
+    p1 = JIJI::Process.new( "1", @dir, agent_mng, @logger, nil, @registry_mock, true )
+    assert_equals p1.id, "1"
+    assert_equals p1.agent_manager, agent_mng
+    assert_equals p1["agents"], agents
+    assert_equals agent_mng.get("aaa"), nil
+    assert_equals agent_mng.get("bbb"), nil
+    assert_equals p1["trade_enable"], nil
+    assert_equals p1.agent_manager.operator.trade_enable, false
+    assert_equals p1.outputs, { "aaa"=>["from_registry"] }
+
+    # RMTでエージェントの追加時にエラー
+    agents << {"id"=>"bbb",
+       "name"=>"テストエージェント4",
+       "class"=>"testclass2@foo.rb",
+       "class_name" => "testclass",
+       "file_name" => "foo.rb",
+       "description" => "説明2aaaaaa",
+       "property_def" => {"id"=>"x", "default"=>85 },
+       "properties" => {"x"=>60, "y"=>41 }}
+    begin
+      p1["agents"] = agents
+      fail
+    rescue
+    end
+
+    # RMTでエージェントのプロパティ更新時にエラー
+    agents2 = [
+      {"id"=>"aaa",
+       "name"=>"テストエージェント1",
+       "class"=>"testclass@foo.rb",
+       "class_name" => "testclass",
+       "file_name" => "foo.rb",
+       "description" => "説明",
+       "property_def" => {"id"=>"x", "default"=>85 },
+       "properties" => {"x"=>50, "y"=>41 }}
+    ]
+    error_registry = RegistryMock.new
+    class << error_registry
+      def create( cl, property )
+        a = Agent.new( cl, property )
+        class << a
+          def properties=(props)
+            raise "test"
+          end
+        end
+        return a
+      end
+    end
+    agent_mng = new_agent_mang(error_registry)
+    p1 = JIJI::Process.new( "2", @dir, agent_mng, @logger, {"agents"=>agents2}, @registry_mock, true )
+    assert_equals p1["agents"], agents2
+    assert_equals agent_mng.get("aaa").agent.cl, "testclass@foo.rb"
+    assert_equals agent_mng.get("aaa").agent.properties, {"x"=>50, "y"=>41 }
+    assert_equals agent_mng.get("bbb"), nil
+    assert_equals p1.outputs, { "aaa"=>["out1","out2"] }
+
+    begin
+      p1["agents"] = [
+        {"id"=>"aaa",
+         "name"=>"テストエージェント1",
+         "class"=>"testclass@foo.rb",
+         "class_name" => "testclass",
+         "file_name" => "foo.rb",
+         "description" => "説明",
+         "property_def" => {"id"=>"x", "default"=>85 },
+         "properties" => {"x"=>50, "y"=>31 }}
+      ]
+      fail
+    rescue
+    end
+    assert_equals p1["agents"], agents2
+    assert_equals agent_mng.get("aaa").agent.cl, "testclass@foo.rb"
+    assert_equals agent_mng.get("aaa").agent.properties, {"x"=>50, "y"=>41 }
+    assert_equals agent_mng.get("bbb"), nil
+    assert_equals p1.outputs, { "aaa"=>["out1","out2"] }
+
+    # バックテストでエージェント生成時にエラー
+    error_registry = RegistryMock.new
+    class << error_registry
+      def create; raise "test"; end
+    end
+    agent_mng = new_agent_mang(error_registry)
+    begin
+      JIJI::Process.new( "1", @dir, agent_mng, @logger, {"agents"=>agents}, @registry_mock, false )
+      fail
+    rescue
+    end
+  end
+
+  # エージェントマネージャを再作成する
+  def new_agent_mang( agent_registory=RegistryMock.new )
+    agent_mng = JIJI::AgentManager.new( "aaa", agent_registory, Logger.new(STDOUT))
+    agent_mng.operator = Struct.new(:trade_enable).new(true)
+    agent_mng.conf = CONF
+    agent_mng.conf.set( [:agent,:safe_level], 0)
+    agent_mng.registry = @registry_mock
+    return agent_mng
+  end
+
   class Agent
     include JIJI::Agent
 
@@ -174,6 +330,9 @@ class ProcessTest <  RUNIT::TestCase
     def properties=( properties )
       @properties = properties
     end
+    def output
+      ["out1","out2"]
+    end
     attr :cl, true
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: unageanu-jiji
 version: !ruby/object:Gem::Version 
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors: 
 - Masaya Yamauchi
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-03-26 00:00:00 -07:00
+date: 2009-04-08 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency