Integrate Contract_Read_ABCI with Learning ABCI to Enhance FSM Workflow #14
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 12 commits
December 1, 2024 04:05
…nents This commit introduces the ERC20_NEW contract package into the contracts folder, featuring the new class ERC20_NEW designed to enhance our ABCI learning agents with updated token handling capabilities. Changes span across several components to ensure comprehensive integration: 1. **Contracts:** - Added ERC20_NEW class in the erc20_new package to manage interactions with ERC20 tokens more effectively. 2. **Agent Configuration:** - Updated �ea-config.yaml to reference the new ERC20_NEW contract, ensuring that the agent recognizes and utilizes this new contract class. 3. **Service Configuration:** - Modified service.yaml to include configurations that align with the changes in the ERC20_NEW contract operations. 4. **Skill Updates:** - Updated skill.yaml in both skill/learning_abci and skill/learning_chained_abci to ensure skills are compatible with the new contract functionalities. - Revised behaviors in �ehaviours.py to incorporate methods from ERC20_NEW for enhanced token transactions. These updates ensure that our learning agents are well-equipped to handle ERC20 token transactions with the latest contract logic, reflecting improved efficiency and compliance with current blockchain standards.
- Extend ConditionalNativeTransferBehaviour to handle conditional deposits into an ERC20 contract. - Add logic to deposit native coins based on the parity of the calculated transaction value. - Update contract.py in erc20_new and behaviours.py in skill/learning_abci with new deposit functionality.
…actions This commit updates the ConditionalNativeTransferBehaviour to bundle native token transfers and ERC20 deposits into a single multisend transaction. It modifies the get_multisend_safe_tx_hash function to accept both native transfer and ERC20 deposit data. This integration allows for more efficient transaction processing and cost optimization by executing both actions in a single blockchain transaction. Changes: - Modified ConditionalNativeTransferBehaviour to support gathering data for native and ERC20 transactions. - Updated get_multisend_safe_tx_hash to handle the packaging of multiple transaction data. - Integrated _build_safe_tx_hash to create the actual multisend transaction from the prepared data.
…th LearningChainedAbci
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
This pull request introduces changes across multiple ABCI skills to enhance the Finite State Machine (FSM) workflow within our blockchain application framework. The key integration point is the linking of the
DecisionMakingRoundfrom thelearning_abciskill directly to theContractDataReadRoundof the newly createdcontract_read_abciskill. This ensures that once decision-making is completed, the system seamlessly transitions to contract data reading, enhancing the data flow and consistency across operations.Changes
skill/learning_chained_abciskill.yamlto include new dependencies and entry points for thecontract_read_abci.fsm_specification.yamlto reflect new state transitions that link to thecontract_read_abci.composition.pyto integrate new skill operations within the ABCI app chain.models.pyandbehaviours.pyto support new logic and state transitions.skill/contract_read_abciinit.py,behaviours.py,modules.py,payloads.py,rounds.py,dialogues.py,handles.py,fsm-specification.yaml) to manage contract data reading processes.fsm-specification.yamlto define clear entry and exit points for contract data interactions.skill/learning_abci/rounds.pyDecisionMakingRoundto trigger transitions into thecontract_read_abciskill upon completion.Configuration and Deployment
agents/aea-config.yamlto include new skill configurations and ensure proper agent setups.