-
Notifications
You must be signed in to change notification settings - Fork 241
Integrate open trader #147
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
Open
skymike
wants to merge
47
commits into
Open-Trader:dev
Choose a base branch
from
skymike:integrate-open-trader
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or 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
This script uploads files to an existing GitHub repository, creating necessary directories and files for a crypto risk dashboard project.
Refactor Dockerfile to use multi-stage build and add health check.
Updated Docker Compose configuration to include health checks for the API and database services, and removed the Redis service dependency from the worker.
Removed streamlit version from requirements.
Refactor code for better readability and structure, including improvements to API base resolution and data fetching functions.
Refactor sentiment fetching to include real API integration and enhance keyword handling.
Added indexes for faster filtering and created a continuous aggregate for 1-hour candles data with a refresh policy.
Deleted the optional Slack notification step on job failure from the crypto-worker GitHub Actions workflow. This simplifies the workflow and removes dependency on the SLACK_WEBHOOK_URL secret.
Removed optional Slack notification step on failure.
Updated cache key for Python packages and removed installation of common requirements.
Replaces usage of 'services.common' with a new 'services_common' namespace package for easier imports. Updates worker script to use repo root for sys.path and removes redundant dependency installation from workflow. Adds services_common/__init__.py to expose common modules under the new namespace.
Added package docstrings to services and services.common. Updated services_common/__init__.py to eagerly import modules and adapters, log import errors for better debugging, and handle exceptions more robustly. Clarified import order in run_worker.py to ensure correct namespace initialization.
Changed all imports from 'services_common' to 'services.common' for consistency and reliability. Updated worker script to remove unnecessary namespace package import and use direct imports from the new namespace.
Added debug statements to verify repo and services paths. Explicitly imported the services package before importing submodules to ensure proper package recognition.
Adds the common directory to sys.path and updates debug output to clarify import paths. Imports from services.common are now wrapped in a try/except block to provide better error visibility during import failures.
Cleaned up unresolved merge conflict markers in requirements.txt to ensure proper dependency management and prevent installation errors.
Resolved merge conflict markers and refactored both the Streamlit UI and sentiment adapter for clarity and maintainability. Improved API base resolution, data fetching, and signal display in the UI. Enhanced sentiment analysis logic and typing in the adapter. Renamed workflow file for consistency.
Resolved merge conflict artifacts in schema_timescale.sql, ensuring proper SQL syntax and clarity. The file now correctly enables the TimescaleDB extension, creates hypertables, adds necessary indexes, and sets up a continuous aggregate with an auto-refresh policy.
…rfilePath and context Changed the build configuration in railway.json to use DOCKERFILE as the builder, added the path to the Dockerfile, and set the context for the build process. This update enhances the deployment setup for the application.
Implemented minimal CSS tweaks for a modern look in the Crypto Risk Dashboard. Added a live market snapshot feature that fetches and displays real-time cryptocurrency prices and changes. Updated the refresh button to clear the market snapshot cache. Improved the extraction of base symbols from trading pairs for better data handling.
Introduced a new POST endpoint for manual data ingestion in the API, allowing background processing of data updates. Enhanced the upsert functionality in the database to handle JSON data types and improved conflict resolution strategies. Updated the UI to include a button for triggering manual ingests, providing users with better control over data updates.
Revised the RAILWAY_DEPLOY.md to clarify the default trading pairs for the SYMBOLS variable, now including the top 30 perpetual pairs. Updated the GitHub Actions workflow to reflect the new default trading pairs. Enhanced the config.py to load a comprehensive list of default symbols. Improved the UI to utilize cached time series data for better performance and added new analytics modules for user insights.
Updated the RAILWAY_DEPLOY.md to include optional Telegram bot configuration for alerts. Enhanced the Streamlit UI to display a global market overview with key metrics such as market cap, volume, and BTC dominance. Added new data fetching functions and improved visualizations for sentiment and funding analysis. Updated the layout for better user experience and added functionality for live liquidation feeds.
Updated the configuration loading in config.py to handle default symbols more effectively, allowing for user-defined symbols while maintaining a comprehensive default list. Improved the asset fetching logic in app.py to include fallback mechanisms for missing data and optimized the parameters for API requests. Adjusted UI layout for better display of market data.
…ions Modified the GitHub Actions workflow to adjust the worker service schedule from every 5 minutes to every 15 minutes. Updated the configuration files to reflect this change and added a new optional `SIGNAL_PROFILE` variable for controlling signal strictness. Enhanced the API to support different profiles in signal computation, improving the flexibility of the signal generation process. Updated the UI to allow users to select their preferred signal profile, enhancing user experience and customization.
Updated the Streamlit UI to include a live time display in Amsterdam timezone, enhancing user experience. Refactored the market snapshot section to utilize a more structured layout with a DataFrame for better data presentation. Improved the styling of signal metrics and added new visualizations for market data, including sentiment trends and volume analysis. Enhanced the overall responsiveness and clarity of the dashboard components.
Updated the presentation of scoring notes in the Hot Signals section of the UI. Changed the layout to use bullet points for better readability and clarity, enhancing the overall user experience.
Added .pyc files generated by Python 3.12 for various modules in the services directory. These files are typically not committed to version control and may have been added unintentionally.
Included a section in the README.md for Linux deployment, providing a reference to a systemd unit and Nginx TLS proxy example located in deploy/linux/README.md. This addition aims to assist users in setting up the application on Linux systems.
|
Integrate open trader |
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.
No description provided.