-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Pearl trader #356
base: chore/reorg
Are you sure you want to change the base?
Update Pearl trader #356
Conversation
diff from |
@Adamantios confirm that the trader_pearl configuration in this PR is appropriate for 0.18.2 / 0.18.7 |
The configs look good from a framework's perspective. Could you elaborate on that more? |
The comment is about if you could check if the pearl_trader.yaml is missing (or has an inconvenient value) for any config params that have been introduced recently, in terms of of tools/multibets/simulation/... etc. |
You could diff with the main service config and use the exact same defaults. I think we could merge the two service files anyway. |
Signed-off-by: OjusWiZard <[email protected]>
Update Pearl trader for the refactored branch on olas-operate.