AbandonedLand / MarketMaker

Chia Market Maker
5 stars 0 forks source link

Error when creating offers #1

Open Drewski241 opened 1 month ago

Drewski241 commented 1 month ago

Hey when i try to run the AMM i get this message just after trying to create offers. "Invalid REQUEST JSON: Expecting property name enclosed in double quotes: line 2 column 5 (char 7)" Also another thing to note the rename cats command should come after the . .\Boot.ps1 command otherwise it won't work. Furthermore I'm very interested in getting this working for MZ token and Dat meme token. Thanks for your help with all this and it's so awesome.

AbandonedLand commented 1 month ago

Are you running PowerShell version 7.3+ ?

Assuming you are running something above 7.3, what trading pairs are enabled in your $config?

 # Traded pairs -  To turn off a pair, change it to $false from $true.
    trading_pair_xch_usdc = $true
    trading_pair_xch_usdcb = $true
    trading_pair_usdc_millieth = $true
    trading_pair_usdcb_milliethb = $true

I may have screwed some things up by adding DBX and HOA to the mix without following the same rules as I did for USDC(b). In the Run-Amm I didn't add checks to see if DBX and HOA are to be traded. I'll get that updated now.

AbandonedLand commented 1 month ago

I'm working on breaking this project up into component pieces with better documentation. I've added https://github.com/AbandonedLand/dexiePowerShell to my public repositories and will do the same with TibetSwap and MintGarden.

Also, take a look at this: https://github.com/AbandonedLand/PSChia/blob/main/grid.ps1 It is a Constant Product market maker that uses TibetSwap to get price information then it creates offers based on a "What if your funds were in Tibetswap also" scenario. But theres a bit of a flaw in that one where if someone takes the smallest offer over and over again, you wont get the increasing price that you would have from the AMM.

AbandonedLand commented 1 month ago

I've uploaded some changes. I've added a check for DBX, DAC (nft), and HOA trading. I've also updated the Rename-CATs function to include HOA. I haven't included the rename function anywhere in the run-amm because I didn't want it to run each time it was executed. But I can add in a check to see if the Wallet contains all the right names and if not run the function. But I'd rather let the user rename their wallets on purpose then trying to do it automatically for them.

Drewski241 commented 1 month ago

HeyI am running powershell 7.4.4 and I think I grabbed the zip file before you added HOA. However at the moment only pair I have as true is usds.Sent from my iPhoneOn Aug 10, 2024, at 6:37 AM, The Mayor of Abandoned Land @.***> wrote: Are you running PowerShell version 7.3+ ? Assuming you are running something above 7.3, what trading pairs are enabled in your $config?

Traded pairs - To turn off a pair, change it to $false from $true.

trading_pair_xch_usdc = $true
trading_pair_xch_usdcb = $true
trading_pair_usdc_millieth = $true
trading_pair_usdcb_milliethb = $true

I may have screwed some things up by adding DBX and HOA to the mix without following the same rules as I did for USDC(b). In the Run-Amm I didn't add checks to see if DBX and HOA are to be traded. I'll get that updated now.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Drewski241 commented 1 month ago

I’m going to try this very soon.Sent from my iPhoneOn Aug 10, 2024, at 7:00 AM, The Mayor of Abandoned Land @.***> wrote: I've uploaded some changes. I've added a check for DBX, DAC (nft), and HOA trading. I've also updated the Rename-CATs function to include HOA. I haven't included the rename function anywhere in the run-amm because I didn't want it to run each time it was executed. But I can add in a check to see if the Wallet contains all the right names and if not run the function. But I'd rather let the user rename their wallets on purpose then trying to do it automatically for them.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>