Skip to content

Commit

Permalink
Remove msgpack for now
Browse files Browse the repository at this point in the history
  • Loading branch information
xmatthias committed Sep 20, 2022
1 parent 4422ac7 commit 3274bb0
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 13 deletions.
2 changes: 1 addition & 1 deletion freqtrade/rpc/api_server/ws/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,5 @@
# isort: off
from freqtrade.rpc.api_server.ws.types import WebSocketType
from freqtrade.rpc.api_server.ws.proxy import WebSocketProxy
from freqtrade.rpc.api_server.ws.serializer import HybridJSONWebSocketSerializer, MsgPackWebSocketSerializer
from freqtrade.rpc.api_server.ws.serializer import HybridJSONWebSocketSerializer
from freqtrade.rpc.api_server.ws.channel import ChannelManager, WebSocketChannel
9 changes: 0 additions & 9 deletions freqtrade/rpc/api_server/ws/serializer.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import logging
from abc import ABC, abstractmethod

import msgpack
import orjson
import rapidjson
from pandas import DataFrame
Expand Down Expand Up @@ -46,14 +45,6 @@ def _deserialize(self, data: str):
return rapidjson.loads(data, object_hook=_json_object_hook)


class MsgPackWebSocketSerializer(WebSocketSerializer):
def _serialize(self, data):
return msgpack.packb(data, use_bin_type=True)

def _deserialize(self, data):
return msgpack.unpackb(data, raw=False)


# Support serializing pandas DataFrames
def _json_default(z):
if isinstance(z, DataFrame):
Expand Down
3 changes: 1 addition & 2 deletions requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,5 @@ python-dateutil==2.8.2
schedule==1.1.0

#WS Messages
websockets~=10.3
msgpack~=1.0.4
websockets==10.3
janus==1.0.0
1 change: 0 additions & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,6 @@
'aiofiles',
'schedule',
'websockets',
'msgpack',
'janus'
],
extras_require={
Expand Down

0 comments on commit 3274bb0

Please sign in to comment.