Skip to content
This repository has been archived by the owner on Apr 19, 2022. It is now read-only.

Reimplement FCF functionality #410

Open
perryhuynh opened this issue Dec 5, 2019 · 4 comments
Open

Reimplement FCF functionality #410

perryhuynh opened this issue Dec 5, 2019 · 4 comments
Assignees

Comments

@perryhuynh
Copy link
Owner

Looks like the FCF functionality still needs to be reimplemented

@mrmin123
Copy link
Collaborator

mrmin123 commented Dec 7, 2019

Are you still running FCF? Please post the combat data from a combat node post-FCF if you can..

@mrmin123
Copy link
Collaborator

mrmin123 commented Dec 7, 2019

And the resulting kcsapi/api_get_member/ship_deck API response.

@mrmin123
Copy link
Collaborator

I've added some logic necessary for FCF, but am waiting on API responses to be able to finalize it.

The changes also have rudimentary support for #94, although it needs update to the config handler and GUI to be fully implemented.

@mrmin123
Copy link
Collaborator

mrmin123 commented Dec 21, 2019

5th ship in escort fleet taiha'ed, retreating with 3rd ship in escort fleet. Relevant portions in json responses are as follows.

kcsapi/api_req_sortie/battleresult; contains api showing 1-based idx for ships to escape and ships that can escort:

{
  "api_data": {
    ...
    "api_escape_flag": 1,
    "api_escape": {
      "api_escape_idx": [
        11
      ],
      "api_tow_idx": [
        9,
        10
      ]
    }
  }
}

kcsapi/api_req_combined_battle/goback_port; might as well be empty:

{"api_result":1,"api_result_msg":"\u6210\u529f"}

kcsapi/api_get_member/ship_deck; retreated ships have fuel value of 0:

{
  "api_data": {
    "api_ship_data": [
      {
        ...
        "api_fuel": 0,
        ...
      }
    ]
  }
}

kcsapi/api_req_combined_battle/battle; no changes to reported HP:

{
  "api_data": {
    ...
    "api_escape_idx_combined": [
      3,
      5
    ],
    ...
  }
}

Outstanding question(s):

  • What is the dict key for the primary fleet equivalent of api_escape_idx_combined?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants