-
Notifications
You must be signed in to change notification settings - Fork 9
Update README.md, and various related changes #845
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
Marc-Andrieu
wants to merge
44
commits into
main
Choose a base branch
from
update/readme
base: main
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
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
3414173
Python 3.11 to 3.12
Marc-Andrieu 945a7ce
About requirements
Marc-Andrieu 39c61d0
About the db
Marc-Andrieu 94b25fd
While I'm touching the .gitignore
Marc-Andrieu fd8f8ac
Unrelated: about pyproject.toml
Marc-Andrieu 48a1527
About the .env
Marc-Andrieu 9c7996c
About config.yaml
Marc-Andrieu 682043b
About launching Hyperion and creating your own admin user
Marc-Andrieu ef0e129
Beyond initial config
Marc-Andrieu 2234476
Fix(.gitignore): distinguish folders for readability
Marc-Andrieu 5cc4a0a
rm requests, use httpx instead, update requirements
Marc-Andrieu c9d1cc2
Type-checking fixes
Marc-Andrieu 536cb44
Feat: async log handler running in a different thread
Marc-Andrieu f27b2b2
re-run tests
Marc-Andrieu bcb94ce
Feat: use the scheduler to synchronically defer writing the logs
Marc-Andrieu 5515ac8
Details feats on config.yaml
Marc-Andrieu 1b5699a
Feat: fastapi's arg "app/main.py" is useless
Marc-Andrieu aa1bc8f
Feat: psycopg[binary] as a common requirement
Marc-Andrieu d10e22c
Feat: n° on great steps
Marc-Andrieu 4fe4ffe
Fix: REDIS_HOST false-ish values to disable it
Marc-Andrieu b5579b9
Test: expandable parts
Marc-Andrieu c514e9c
Minor completions
Marc-Andrieu 262bc4c
YAML inline comments; mv CORS_ORIGINS up to auth section
Marc-Andrieu a343478
Test: use a .gitguardian.yaml
Marc-Andrieu 5904cba
use FACTORIES_DEMO_USERS; rework README with that; Factories and Auth…
Marc-Andrieu 04f935a
Fix(HA): no error logs in dev when it's alright
Marc-Andrieu 75f4b87
rm useless gitguardian.yml, leave example credentials in clear
Marc-Andrieu e3e4063
Fix: test config
Marc-Andrieu be1433b
Fix test YAML config
Marc-Andrieu c2217f7
Failing tests: try previous psycopg minor
Marc-Andrieu 8814f13
Fix: rm factories from tests
Marc-Andrieu 25e52ef
hopelessness
Marc-Andrieu 948a8ef
Fixes for test workflows and db
Marc-Andrieu ce00e05
Minor fixes
Marc-Andrieu 4bb9133
Minor fixes
Marc-Andrieu fa25af8
small rewording
Marc-Andrieu 3874df6
Put back `requests` and synchronous Matrix logging
Marc-Andrieu 4c486c5
Fix ruff: TCH was renamed TC in 0.8
Marc-Andrieu 2bbfe2c
Fix mypy too
Marc-Andrieu f819dfb
Numerous fixes due to the comments
Marc-Andrieu 894b65b
Rename to requirements.txt
Marc-Andrieu 39a31d3
Fix: make profile pictures imported in Synapse again (#881)
Marc-Andrieu 731e58f
Merge branch 'main' into update/readme
Marc-Andrieu 8bba57b
Update .env.template
Rotheem File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,11 +3,14 @@ | |
| ############################ | ||
|
|
||
| # Should be set to the name of the postgres container | ||
| POSTGRES_HOST="hyperion-db" | ||
| POSTGRES_USER="" | ||
| POSTGRES_PASSWORD="" | ||
| # Should be set to the name of the postgres container | ||
| POSTGRES_HOST="" | ||
| POSTGRES_DB="hyperion" | ||
| POSTGRES_TZ="Etc/UTC" | ||
| # The database can be accessed through the command line with | ||
| # psql -U $POSTGRES_USER -d $POSTGRES_DB | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. the -d $POSTGRES_DB is useless
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It's useless if you want to connect to the dbms, not if you want to connect directly to the db for Hyperion |
||
|
|
||
| ######################## | ||
| # Redis configuration # | ||
|
|
@@ -16,8 +19,8 @@ POSTGRES_TZ="Etc/UTC" | |
| # We use the default redis configuration, so the protected mode is enabled by default (see https://redis.io/docs/manual/security/#protected-mode) | ||
| # If you want to use a custom configuration, a password and a specific binds should be used to avoid security issues | ||
|
|
||
| # May be left at "" during dev if you don't have a redis server running, in production it should be set to the name of the redis container | ||
| REDIS_HOST="hyperion-redis" | ||
| # REDIS_HOST may be commented to disable Redis during development if you don't have a redis server running, in production it should be set to the name of the redis container | ||
| #REDIS_HOST="localhost" | ||
| REDIS_PORT=6379 | ||
| # Should be commented during development to work with docker-compose-dev, and set in production | ||
| #REDIS_PASSWORD="" | ||
|
|
||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -26,7 +26,7 @@ jobs: | |
| uses: actions/[email protected] | ||
| with: | ||
| path: ~/.cache/uv | ||
| key: ${{ runner.os }}-python-${{ steps.setup-python.outputs.python-version }}-uv-${{ hashFiles('requirements-common.txt', 'requirements-dev.txt') }} | ||
| key: ${{ runner.os }}-python-${{ steps.setup-python.outputs.python-version }}-uv-${{ hashFiles('requirements.txt', 'requirements-dev.txt') }} | ||
|
|
||
| - name: Install uv | ||
| run: curl -LsSf https://astral.sh/uv/install.sh | sh | ||
|
|
||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -32,7 +32,7 @@ jobs: | |
| - 6379:6379 | ||
| postgres: | ||
| # Docker Hub image | ||
| image: postgres | ||
| image: "postgres:15.1" | ||
| # Provide the password for postgres | ||
| env: | ||
| POSTGRES_PASSWORD: "somerealpassword" | ||
|
|
@@ -57,14 +57,14 @@ jobs: | |
| uses: actions/setup-python@v6 | ||
| id: setup-python | ||
| with: | ||
| python-version: "3.11" | ||
| python-version: "3.12" | ||
|
|
||
| - name: Cache uv folder | ||
| id: cache-uv | ||
| uses: actions/[email protected] | ||
| with: | ||
| path: ~/.cache/uv | ||
| key: ${{ runner.os }}-python-${{ steps.setup-python.outputs.python-version }}-uv-${{ hashFiles('requirements-common.txt', 'requirements-dev.txt') }} | ||
| key: ${{ runner.os }}-python-${{ steps.setup-python.outputs.python-version }}-uv-${{ hashFiles('requirements.txt', 'requirements-dev.txt') }} | ||
|
|
||
| - name: Install uv | ||
| run: curl -LsSf https://astral.sh/uv/install.sh | sh | ||
|
|
||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,39 +1,36 @@ | ||
| # Cache folders | ||
| __pycache__ | ||
| .mypy_cache | ||
| .pytest_cache | ||
| .ruff_cache | ||
|
|
||
| # Databases and data | ||
| sql_app.db | ||
| test.db | ||
| __pycache__/ | ||
| .mypy_cache/ | ||
| .pytest_cache/ | ||
| .ruff_cache/ | ||
|
|
||
| # Databases (PostgreSQL and SQLite) | ||
| hyperion-db/ | ||
| *.db | ||
|
|
||
| # Persistent data as files | ||
| data/ | ||
|
|
||
| # Dotenv file | ||
| # Configuration and secrets files | ||
| .env | ||
| config.yaml | ||
|
|
||
|
|
||
| # Firebase secrets | ||
| firebase.json | ||
|
|
||
| # macOS | ||
| .DS_Store | ||
|
|
||
| # Virtual environment | ||
| .venv | ||
| .python-version | ||
| .venv/ | ||
|
|
||
| # Logs | ||
| logs/ | ||
|
|
||
| # Pytest-cov | ||
| .coverage | ||
|
|
||
| # Jinja templates test output | ||
| # Jinja output templates and test output | ||
| tests/jinja_test_outputs/ | ||
| node_modules/ | ||
|
|
||
| # Local testing | ||
| scripts/ | ||
|
|
||
| node_modules |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| 3.12.11 |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,8 @@ | ||
| { | ||
| "recommendations": [ | ||
| "ms-python.python", | ||
| "ms-python.mypy-type-checker", | ||
| "charliermarsh.ruff", | ||
| ] | ||
| } | ||
| "recommendations": [ | ||
| "ms-python.python", | ||
| "ms-python.mypy-type-checker", | ||
| "charliermarsh.ruff", | ||
| "tamasfe.even-better-toml" | ||
| ] | ||
| } |
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 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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.