mirror of
https://github.com/EDCD/EDMarketConnector.git
synced 2025-04-22 11:50:28 +03:00
Merge pull request #625 from A-UNDERSCORE-D/fix/624-inara-too-fast
Switched to using a timer for inara updates
This commit is contained in:
commit
525ebae61d
@ -4,6 +4,7 @@
|
||||
|
||||
from collections import OrderedDict
|
||||
import json
|
||||
from typing import Any, Union
|
||||
import requests
|
||||
import sys
|
||||
import time
|
||||
@ -26,7 +27,7 @@ FAKE = ['CQC', 'Training', 'Destination'] # Fake systems that shouldn't be sent
|
||||
CREDIT_RATIO = 1.05 # Update credits if they change by 5% over the course of a session
|
||||
|
||||
|
||||
this = sys.modules[__name__] # For holding module globals
|
||||
this: Any = sys.modules[__name__] # For holding module globals
|
||||
this.session = requests.Session()
|
||||
this.queue = Queue() # Items to be sent to Inara by worker thread
|
||||
this.lastlocation = None # eventData from the last Commander's Flight Log event
|
||||
@ -49,8 +50,11 @@ this.loadout = None
|
||||
this.fleet = None
|
||||
this.shipswap = False # just swapped ship
|
||||
|
||||
this.last_update_time = time.time() # last time we updated (set to now because we're going to update quickly)
|
||||
FLOOD_LIMIT_SECONDS = 45 # minimum time between sending non-major cargo triggered messages
|
||||
# last time we updated, if unset in config this is 0, which means an instant update
|
||||
LAST_UPDATE_CONF_KEY = 'inara_last_update'
|
||||
# this.last_update_time = config.getint(LAST_UPDATE_CONF_KEY)
|
||||
FLOOD_LIMIT_SECONDS = 30 # minimum time between sending events
|
||||
this.timer_run = True
|
||||
|
||||
|
||||
# Main window clicks
|
||||
@ -85,6 +89,10 @@ def plugin_start3(plugin_dir):
|
||||
this.thread = Thread(target = worker, name = 'Inara worker')
|
||||
this.thread.daemon = True
|
||||
this.thread.start()
|
||||
|
||||
this.timer_thread = Thread(target=call_timer, name='Inara timer')
|
||||
this.timer_thread.daemon = True
|
||||
this.timer_thread.start()
|
||||
return 'Inara'
|
||||
|
||||
def plugin_app(parent):
|
||||
@ -96,11 +104,14 @@ def plugin_app(parent):
|
||||
def plugin_stop():
|
||||
# Send any unsent events
|
||||
call()
|
||||
time.sleep(0.1) # Sleep for 100ms to allow call to go out, and to force a context switch to our other threads
|
||||
# Signal thread to close and wait for it
|
||||
this.queue.put(None)
|
||||
this.thread.join()
|
||||
this.thread = None
|
||||
|
||||
this.timer_run = False
|
||||
|
||||
def plugin_prefs(parent, cmdr, is_beta):
|
||||
|
||||
PADX = 10
|
||||
@ -188,7 +199,7 @@ def journal_entry(cmdr, is_beta, system, station, entry, state):
|
||||
|
||||
# Send any unsent events when switching accounts
|
||||
if cmdr and cmdr != this.cmdr:
|
||||
call()
|
||||
call(force=True)
|
||||
|
||||
this.cmdr = cmdr
|
||||
this.FID = state['FID']
|
||||
@ -245,8 +256,6 @@ def journal_entry(cmdr, is_beta, system, station, entry, state):
|
||||
|
||||
if config.getint('inara_out') and not is_beta and not this.multicrew and credentials(cmdr):
|
||||
try:
|
||||
old_events = len(this.events) # Will only send existing events if we add a new event below
|
||||
|
||||
# Dump starting state to Inara
|
||||
|
||||
if (this.newuser or
|
||||
@ -305,6 +314,8 @@ def journal_entry(cmdr, is_beta, system, station, entry, state):
|
||||
|
||||
this.loadout = make_loadout(state)
|
||||
add_event('setCommanderShipLoadout', entry['timestamp'], this.loadout)
|
||||
|
||||
call() # Call here just to be sure that if we can send, we do, otherwise it'll get it in the next tick
|
||||
|
||||
|
||||
# Promotions
|
||||
@ -434,35 +445,21 @@ def journal_entry(cmdr, is_beta, system, station, entry, state):
|
||||
|
||||
cargo = [OrderedDict([('itemName', k), ('itemCount', state['Cargo'][k])]) for k in sorted(state['Cargo'])]
|
||||
|
||||
# if our cargo differers from last we checked, we're at a station,
|
||||
# and our flood limit isnt covered, queue an update
|
||||
should_poll = this.cargo != cargo and time.time() - this.last_update_time > FLOOD_LIMIT_SECONDS
|
||||
|
||||
# Send event(s) to Inara
|
||||
if entry['event'] == 'ShutDown' or len(this.events) > old_events or should_poll:
|
||||
|
||||
# Send cargo and materials if changed
|
||||
if this.cargo != cargo:
|
||||
add_event('setCommanderInventoryCargo', entry['timestamp'], cargo)
|
||||
this.cargo = cargo
|
||||
materials = []
|
||||
for category in ['Raw', 'Manufactured', 'Encoded']:
|
||||
materials.extend([ OrderedDict([('itemName', k), ('itemCount', state[category][k])]) for k in sorted(state[category]) ])
|
||||
if this.materials != materials:
|
||||
add_event('setCommanderInventoryMaterials', entry['timestamp'], materials)
|
||||
this.materials = materials
|
||||
|
||||
# Queue a call to Inara
|
||||
call()
|
||||
# Send cargo and materials if changed
|
||||
if this.cargo != cargo:
|
||||
add_event('setCommanderInventoryCargo', entry['timestamp'], cargo)
|
||||
this.cargo = cargo
|
||||
materials = []
|
||||
for category in ['Raw', 'Manufactured', 'Encoded']:
|
||||
materials.extend([ OrderedDict([('itemName', k), ('itemCount', state[category][k])]) for k in sorted(state[category]) ])
|
||||
if this.materials != materials:
|
||||
add_event('setCommanderInventoryMaterials', entry['timestamp'], materials)
|
||||
this.materials = materials
|
||||
|
||||
except Exception as e:
|
||||
logger.debug('Adding events', exc_info=e)
|
||||
return str(e)
|
||||
|
||||
#
|
||||
# Events that don't need to be sent immediately but will be sent on the next mandatory event
|
||||
#
|
||||
|
||||
# Send credits and stats to Inara on startup only - otherwise may be out of date
|
||||
if entry['event'] == 'LoadGame':
|
||||
add_event('setCommanderCredits', entry['timestamp'],
|
||||
@ -856,14 +853,23 @@ def add_event(name, timestamp, data):
|
||||
('eventData', data),
|
||||
]))
|
||||
|
||||
def call_timer(wait=FLOOD_LIMIT_SECONDS):
|
||||
while this.timer_run:
|
||||
time.sleep(wait)
|
||||
print(f"INARA: {time.asctime()} TICK")
|
||||
if this.timer_run: # check again in here just in case we're closing and the stars align
|
||||
call()
|
||||
|
||||
# Queue a call to Inara, handled in Worker thread
|
||||
def call(callback=None):
|
||||
def call(callback=None, force=False):
|
||||
if not this.events:
|
||||
return
|
||||
|
||||
this.last_update_time = time.time()
|
||||
|
||||
if (time.time() - config.getint(LAST_UPDATE_CONF_KEY)) <= FLOOD_LIMIT_SECONDS and not force:
|
||||
return
|
||||
|
||||
config.set(LAST_UPDATE_CONF_KEY, int(time.time()))
|
||||
print(f"INARA: {time.asctime()} sending {len(this.events)} entries to inara (call)")
|
||||
data = OrderedDict([
|
||||
('header', OrderedDict([
|
||||
('appName', applongname),
|
||||
@ -886,6 +892,8 @@ def worker():
|
||||
else:
|
||||
(url, data, callback) = item
|
||||
|
||||
print(f"INARA: {time.asctime()} sending {len(data['events'])} entries to inara (worker)")
|
||||
|
||||
retrying = 0
|
||||
while retrying < 3:
|
||||
try:
|
||||
|
Loading…
x
Reference in New Issue
Block a user