Skip to content
GitLab
Select Git revision
This project is mirrored from https://github.com/osmosis-labs/osmosis.git. Pull mirroring updated .
  1. 28 Feb, 2023 4 commits
  3. 27 Feb, 2023 2 commits
  5. 24 Feb, 2023 5 commits
  7. 23 Feb, 2023 9 commits
    • Aidan Salzmann's avatar
      migrate stXXX/XXX constant product pool -> stableswap pool (#4384) · 0355d275
      Aidan Salzmann authored 
      
* migrate stXXX/XXX constant product pool -> stableswap pool

* update upgrade unittest

* send LP tokens to lpWallet

* update tests

* add join, exit unittest

* add checks per comments

* one more comment

* lp pre-upgrade

* clean up and TODOs

* fix e2e

* add query

* rm TestMigrateBalancerToStable, update ExitPool in upgrade_test.go

* rm TestMigrateBalancerToStable

* update changelog

* fix e2e

* more assertions for TestMigrateBalancerToStablePools

* lint

---------

Co-authored-by: default avatarRoman <roman@osmosis.team>
      0355d275
    • Sishir Giri's avatar
      Re[CL:Incentives] Add further tests to uptime accumulator update logic (#4370) · dd769809
      Sishir Giri authored 
      * all test

* added extra test

* fixed comments
      dd769809
    • alpo's avatar
      [CL Incentives]: Sync global uptime accumulators upon liquidity change (#4307) · 146b800f
      alpo authored 
      
* initial push

* push

* untracked files

* add untracked files

* merge branches

* add changes required for fees

* lint

* fix test errors

* clean up

* remove print lines

* feat: single fee accum (#4116)

* single fee accum

* lint

* no longer need to get all positions for fee

* incentive accum initialization

* lint

* Update x/concentrated-liquidity/store_test.go

Co-authored-by: default avatarSishir Giri <sis1001@berkeley.edu>

* add lower level tests and make uptime accum access private

* lint

* add gotests for helper fn

* Update test to use existing variable

Co-authored-by: default avatarRoman <roman@osmosis.team>

* create or update records upon position creation/update

* add tests and clean up

* sync position-related changes

* add support for negative liquidity delta

* add and test helper & set up tick initialization logic

* implement accumulator updates on tick crossing and clean up proto

* clean up comments

* minor test cleanup

* ensure accum initalization follows conventions and clean up tests

* update pool proto and CL pool extension for tracking liq changes

* update new field at pool initialization and add tests

* comment todos

* clean up naming and move empty options to global var

* add clarifying comments and lint

* implement IncentiveRecord proto

* add comments

* lint

* set up wiring for proto

* implement updateUptimeAccumulatorsToNow

* implement and test calcAccruedIncentivesForAccum

* add tests for new helpers

* minor comment updates

Co-authored-by: default avatarRoman <roman@osmosis.team>

* accum test updates

* lint

* clean up diff

* lint

* update go.mod for e2e

* clean up proto and add error message

* lint

* sketch out testing approach

* set up keys

* clean up diff

* clean up comments

* lint

* implement set and get for incentive records

* comment cleanup

* minor comment and test cleanup

* clean up tests and comments

* [CL Incentives] Implement `setIncentiveRecords` and `getIncentiveRecords` (#4283)

* set up keys

* implement set and get for incentive records

* comment cleanup

* minor comment and test cleanup

* Revert "[CL Incentives] Implement `setIncentiveRecords` and `getIncentiveRecords` (#4283)" (#4294)

This reverts commit de7c1467

.

* set incentive records in new position tests

* fix merge conflicts

* add check for init position record values

* fix and finalize uptime accum position tests

* fix go mod for e2e

* clean up diff

* add err check to accum update

* fix rounding error

* go mod tidy

* fix conflicts

* fix e2e

* update go mod

* remove redundant values from incentive record state

* cleanup from review

---------

Co-authored-by: default avatarAdam Tucker <adamleetucker@outlook.com>
Co-authored-by: default avatarAdam Tucker <adam@osmosis.team>
Co-authored-by: default avatarSishir Giri <sis1001@berkeley.edu>
Co-authored-by: default avatarRoman <roman@osmosis.team>
      146b800f
    • Sishir Giri's avatar
      [CL:Test] Add sqrtPrice check to swaps_test (#4201) · 52965de3
      Sishir Giri authored 
      * added sqrt check

* Added all wolfram

* nit

* rebased

* work

* go work sum fix

* rebased

* rebase fix

* fixed some fees

* fixed test

* added checks

* fixed

* fix gosum

* fixed conflicts

* Update x/concentrated-liquidity/swaps_test.go

* Update x/concentrated-liquidity/swaps_test.go
      52965de3
    • Hieu Vu's avatar
      Fix poolmanager query (#4403) · 46530dc1
      Hieu Vu authored 
      * add NewQuerier func

* poolmanager regis query server

* lint

* check query not alter state
      46530dc1
    • Roman's avatar
      fix(CL): swap in given out fees and tests (#4385) · f3c2e1f2
      Roman authored
      f3c2e1f2
    • Nicolas Lara's avatar
      Added rate limits in upgrade (#4340) · a1e2b3d4
      Nicolas Lara authored 
      * added rate limits in upgrade

* added upgrade test

* refactor rate limiting tests and added rate limits pre-upgrade

* lint

* goimports

* experiments for E2E

* post merge lints

* added RL before upgrade

* just get the latest contract

* experiments

* updated wasm file

* remove unnecessary prints

* bad check

* experiments without e2e

* removed unnecessary fmt

* param space initialization for rate limiting (#4360)

* param space initialization for rate limiting

* lint

* better test

* properly initialize empty params

* fix typo

* adding RL to E2E

* removing RL again from E2E

* debug

* genesis

* genesis 2

* add genesis and solve params bug

* uncomment setup rate limiting

* added configurable gov module

* clean up

* clean up and genesis test

* upgrade handler

* genesis clean up

* clean up e2e

* wire querier

* route fix

* fix router

* debug things

* fix init genesis bug

* push fix

* lint

* fix

* clean up upgrades

* testing propper params after upgrade in E2E

* "properly" unmarshaling the parm

* goimports

* allowing for types other than objects in cosmwasm queries

* fix check for param reset

* remove unnecessary prints

* compiled with the proper vesion of workspace optimizer

* Revert "compiled with the proper vesion of workspace optimizer"

This reverts commit ab5cf6cd

.

* added length check

* params experiment for test with state-export

---------

Co-authored-by: default avatarRoman <roman@osmosis.team>
Co-authored-by: default avatarRoman <ackhtariev@gmail.com>
      a1e2b3d4
    • Nicolas Lara's avatar
      Remove unnecessary param from XCS (#4390) · 6232cb9e
      Nicolas Lara authored 
      * removing swap_amount param

* remove bad import

* compiled wasm

* compiled with the proper vesion of workspace optimizer
      6232cb9e
    • Matt, Park's avatar
      Tokenfactory: Add Before send hooks (#4382) · ad1a75be
      Matt, Park authored 
      * Add before send hoooks

* Rename query

* Fix lint

* Nicolas review
      ad1a75be
  9. 22 Feb, 2023 3 commits
  11. 21 Feb, 2023 1 commit
  13. 20 Feb, 2023 9 commits
  15. 17 Feb, 2023 6 commits
    • Roman's avatar
      test(CL Fees): updateFeeGrowthGlobal (#4198) · a5bbf83e
      Roman authored 
      * test(CL Fees): computeFeeChargePerSwapStep function

* move export

* TestUpdateFeeGrowthGlobal

* conflicts
      a5bbf83e
    • Ruslan Akhtariev's avatar
      [e2e][CL]: Withdraw Position (#4120) · 85803982
      Ruslan Akhtariev authored 
      
* withdraw position cmd

* WithdrawPosition: frozenUntil

* WithdrawPosition: little type fix

* wip

* refactoring

* testing

* withdraw position to e2e

* remove testing print

* rename back the CL test

* remove unrelated docs

---------

Co-authored-by: default avatarRoman <roman@osmosis.team>
      85803982
    • Roman's avatar
      docs(CL): begin converting `architecture.md` into module spec (#4363) · 13e7b0c2
      Roman authored 
      
* docs(CL): begin converting `architecture.md` into module spec

* Update x/concentrated-liquidity/architecture.md

Co-authored-by: default avatarAdam Tucker <adam@osmosis.team>

* Update x/concentrated-liquidity/architecture.md

Co-authored-by: default avatarAdam Tucker <adam@osmosis.team>

---------

Co-authored-by: default avatarAdam Tucker <adam@osmosis.team>
      13e7b0c2
    • Roman's avatar
      refactor(CL): MsgCollectFeesResponse field from TokenOut to CollectedFees (#4362) · b6105638
      Roman authored 
      * refactor(CL): MsgCollectFeesResponse field from TokenOut to CollectedFees

* fix proto and regen go mod
      b6105638
    • Adam Tucker's avatar
      fix: CL accumulator and tick initialization logic (#4265) · 631a03bd
      Adam Tucker authored 
      * fix(CL): fix swap and fee problems in compute swap step

* lint

* fix scripts

* clean up Python scripts

* clean up scripts more

* clean up prefix

* spec update

* fix computeFeeChargePerSwapStepOutGivenIn

* improve swap step comments

* lint

* TODO comments

* address TODOs / strategy tests

* TestGetAmountRemainingLessFee

* add logs

* remove print

* initial push

* upgrade osmoutils tag

* update accum after claiming rewards

* error check

* functional fees

* Revert "functional fees"

This reverts commit 111a69bf

.

* additional fixes from call

* Update x/concentrated-liquidity/fees.go

Co-authored-by: default avatarRoman <roman@osmosis.team>

* merge main

* add preparePositionAccumulator with tests

* cover new tick logic with test

* edit correct calc func

* tests: functional test for CL fees (#4282)

* fee functional test

* fix/test(CL Swaps): fix cross tick logic, re-estimate tests; final changes for functional tests (#4301)

* extract loop to helper func

* add noUSDC/ETHExpected const

* parameterize numPositons by type

* add numSwaps to positions struct

---------

Co-authored-by: default avatarRoman <roman@osmosis.team>

---------

Co-authored-by: default avatarRoman <roman@osmosis.team>
      631a03bd
    • ducnt131's avatar
      chore(e2e): upgrade e2e Hermes to v1.2.0 (#4337) · e0bede0b
      ducnt131 authored 
      * Update hermes version for e2e

* Update hermes command

* Update hermes command

* Update hermes command

* config file specifies deprecated setting

* hermes create channel update

* hermes create channel update

* update some argument

* Update success string to determine success
      e0bede0b
  17. 16 Feb, 2023 1 commit
    • Jeremy Liu's avatar
      Protorev smarter logic and more testing (#4181) · 23b13a1d
      Jeremy Liu authored 
      
* Add generalized tests

Added:
- 4 pool mainnet route test
- 2 pool mainnet route test
- Non-osmo/atom denom test

* Add stableswap doomsday test

- Currently panics due to solveCFMMBinarySearchMulti in stableswap calculation

* Add tests for pool point limits

- Tests that the per tx limit works properly
- Tests that the per block limit works properly

* Add pre-check before binary search

- Adds a pre-check before the binary search that tells us if we need to run the binary search at all
- Reduces overall computation/time by avoiding binary searching profit amounts over routes without any profit opportunity

* Add denom to make test pass

- Added "test/3" denom to test non-osmo/atom denoms in previous PR
- didn't notice it broke one test that expected us to only have 2 denoms, so adding "test/3" here so everything passes

* Implement minimum change to have smarter binary search bounds

- This implements a way for us to have a binary search bound that can increase in size to account for large trades above the default range
- I believe this is the minimum-amount-of-new-code approach, but may not be the optimal approach (what this method avoids is having to save the amount in or amount out of the original swap, and then backtracking/converting that amount to the base denom for an arb, and creating bounds off of that)

* Switch range increasing logic

- Doubling isn't the wanted behavior (only wanted if lower bound is 1), just increasing bound by the same range size is wanted (so adding MaxInputAmount achieves this)

* Add logic to extend search bounds when finding optimal amount in

- Increases max iterations to 17 to allow for situation when we need to increase the upper bound
- Add new ExtendedMaxInputAmount variable for us to use as this new max bound range
- Replace bound changing logic from iteratively changing the range to immediately giving our max range

* Move range extension into it's own helper function

* basic benchmark testing for posthandler and epoch hook

* Add SwapAmountOut Test

* Add extended range test

- Tests binary search range extension logic works properly

* Add panic catching test

- This currently fails, this is on purpose so we don't merge the PR and think we are good until this passes and the panics are handled properly

* dynamic step size

* pool points only incremented if profitable

* adding sanity checks for pool point calcs, nits

* Update doomsday testing accounting for refund system

- Makes pool 41 reserves to have an arb opportunity in the doomsday routes
- Changes tx limit and block limit specifically for doomsday testing

* Return nil for no profit opportunity

* adding E2E, removing atom as a base denom, more testing for post handler

* find max profit test fix

* nit

* backporting version tag to 14.x

* comment update for protorev admin account

---------

Co-authored-by: default avatarDavid Terpay <david.terpay@gmail.com>
      23b13a1d