Skip to content
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

Improve Ogmios-Kupo Backend Queries and VyFi DEX Integration #102

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

bhatt-deep
Copy link
Collaborator

@bhatt-deep bhatt-deep commented Sep 14, 2024

This PR enhances the Ogmios Backend pool utxos queries and VyFi DEX integration and improves overall code quality across multiple files. Key changes include:

VyFi DEX Enhancements (src/charli3_dendrite/dexs/amm/vyfi.py)

  • Refactored VyFiCPPState class for better performance and maintainability
  • Implemented asset-based pool selection
  • Improved pools refresh mechanism with error handling

Backend Improvements (src/charli3_dendrite/backend/ogmios_kupo/__init__.py)

  • Updated get_pool_utxos and get_pool_in_tx methods to use payment credentials
  • Improved handling of asset filtering in Kupo requests

Example Script Updates (example.py)

  • Added support for multiple DEX implementations

@bhatt-deep bhatt-deep added the enhancement New feature or request label Sep 14, 2024
@bhatt-deep bhatt-deep self-assigned this Sep 14, 2024
@bhatt-deep bhatt-deep changed the title Optimize ogmios queries Improve Ogmios-Kupo Backend Queries and VyFi DEX Integration Sep 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant