Skip to content

(all-in-one); latest: Startup argument swap as requested in #57#59

Open
comfy-home wants to merge 15 commits into
TheCarBun:mainfrom
comfy-home:startup-argument-swap
Open

(all-in-one); latest: Startup argument swap as requested in #57#59
comfy-home wants to merge 15 commits into
TheCarBun:mainfrom
comfy-home:startup-argument-swap

Conversation

@comfy-home

Copy link
Copy Markdown
Contributor

Merge either this PR, or #57
I'll send you my reply into #57, please, read before merging (then discard the one you decided not to use)


What this PR does:

all discussed changes + what you asked for:

streamlit run app.py → starts local server with the same behaviour as your deployed version (without automatic load of user stats - it's empty)

streamlit run app.py -- mystats → starts local server with pre-filled credentials from env files

---

* Token parsed automatically, and user name pre-filled.

* add `username = "your-git-username"` to secrets.toml
fix: correct contribution count calculation in process_contribution_data
---

start-up commands:

`streamlit run app.py`
* checks your own stats (backend passes your creds from secrets.toml)

`streamlit run app.py -- inspect`
* start server in inspect mode, so user can view stats of any GitHub user. It's not auto-opened in default browser

`streamlit run app.py --server.headless false -- inspect`
* opens dashboard in default browser in inspect mode, so user can view stats of any GitHub user
ENH: inspect mode for viewer functionality
@comfy-home

Copy link
Copy Markdown
Contributor Author

*edit: just updated README file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant