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

Update README.md to specify bids_dir parameter in Streamlit example #72

Merged
merged 1 commit into from
Jun 28, 2024

Conversation

DebuggerDan
Copy link
Contributor

@DebuggerDan DebuggerDan commented Jun 28, 2024

This is a simple documentation-specific pull request for README.md.

Specifically, the further specification of how Streamlit should be run, with a bids_dir argument being passed & notes a possible error regarding non-initialized bids_dir data whilst using Streamlit.

Specifically, the following simple changes are proposed for README.md:
streamlit run src/b2aiprep/app/Dashboard.py
to
streamlit run src/b2aiprep/app/Dashboard.py [path to BIDS directory]

Lastly, a note was added regarding a possible bug encountered whilst attempting to use Streamlit:
After the Streamlit dashboard opens, please wait for the BIDS data to be loaded while on the Dashboard page, as an error regarding a non-initialized bids_dir directory may pop up if the BIDS data has not been fully loaded.

Specifies how streamlit should be ran, with a bids_dir argument being passed & notes a possible error regarding non-initialized bids_dir data whilst using Streamlit
@alistairewj
Copy link
Collaborator

Ah yes, great fix, thank you!

@alistairewj alistairewj merged commit b50823e into sensein:main Jun 28, 2024
2 checks passed
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.

2 participants