Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
161 commits
Select commit Hold shift + click to select a range
a47e05e
test: add initial GUI test
jcharkow Nov 13, 2024
97a4126
add test_file_upload
jcharkow Dec 6, 2024
a188904
add vim tmp files to .gitignore
jcharkow Dec 6, 2024
87e82c5
progress add view tests
jcharkow Dec 6, 2024
b035ac1
Merge branch 'main' into add_tests
jcharkow Dec 6, 2024
99c30a6
add more specific tests for run_workflow
jcharkow Dec 6, 2024
e1cc1db
add ci test with gh actions
jcharkow Dec 6, 2024
31f9a55
fix ci file
jcharkow Dec 11, 2024
bfaa0d4
add pytest to ci file
jcharkow Dec 11, 2024
c320944
add docker compose to template
t0mdavid-m Jan 2, 2025
f32dbd2
newGUID build full app
Arslan-Siraj Jan 2, 2025
45b6e1b
newGUID build with embeddable pyth
Arslan-Siraj Jan 2, 2025
7cd0a41
Merge pull request #125 from t0mdavid-m/add_docker_compose
t0mdavid-m Jan 2, 2025
749584a
Add info for processor architectures.
t0mdavid-m Jan 5, 2025
3b26eaf
Fix typo
t0mdavid-m Jan 5, 2025
b50a587
Merge pull request #126 from Arslan-Siraj/development-wix
t0mdavid-m Jan 15, 2025
dfdd384
test: update run workflow test
jcharkow Jan 22, 2025
db3be5b
test: add comment to spectrum visualization
jcharkow Jan 22, 2025
86f01df
test: try to add spectrum plot test
jcharkow Jan 22, 2025
57a4343
Merge branch 'main' into add_tests
jcharkow Jan 22, 2025
fc80ccd
test: fix bugs
jcharkow Jan 22, 2025
548feaa
add citation section in readme
t0mdavid-m Jan 30, 2025
ca1771f
update citations in parameter summary
t0mdavid-m Jan 30, 2025
de51dd9
temp switcheroo
t0mdavid-m Feb 14, 2025
5104123
fix mamba env
t0mdavid-m Feb 14, 2025
346c934
temp switcheroo 2
t0mdavid-m Feb 14, 2025
d701e1e
Merge pull request #134 from OpenMS/fix_dockerfile_simple
t0mdavid-m Feb 15, 2025
98f0d9f
drop spectra without peaks in data viewer
axelwalter Feb 19, 2025
14308e7
Add workspace management features and enable workspace setting
definitelynotchirag Feb 28, 2025
a1df75c
Refactor workspace checks to simplify conditionals and improve readab…
definitelynotchirag Feb 28, 2025
c17cc66
Merge pull request #94 from jcharkow/add_tests
jcharkow Mar 3, 2025
452c572
Merge pull request #131 from t0mdavid-m/update_citations
jcharkow Mar 3, 2025
362adef
Merge branch 'main' into main
definitelynotchirag Mar 3, 2025
33aa04c
Added installer as a release asset #139
Mar 4, 2025
dc3d146
docker files updated
Mar 4, 2025
3535efd
Adding open ms installation via conda
subCode321 Mar 4, 2025
7e5fbb2
Rename workspace setting key to "enable_workspaces"
definitelynotchirag Mar 4, 2025
7afb63a
Merge branch 'main' of https://github.com/definitelynotchirag/streaml…
definitelynotchirag Mar 4, 2025
431c6e9
fix pyopenms_viz version
jcharkow Mar 4, 2025
f49eaf1
Merge branch 'main' of github.com:OpenMS/streamlit-template
jcharkow Mar 4, 2025
d8e94cd
Update pyopenms_viz to 1.0.0 and fix breaking changes
achalbajpai Mar 4, 2025
19f131c
remove python setup and replace with conda
subCode321 Mar 4, 2025
fa5b563
activate conda env
subCode321 Mar 4, 2025
db30548
fix installation
subCode321 Mar 4, 2025
ca182a0
fix pyopenms version attempt to get tests to pass
jcharkow Mar 4, 2025
1a630bf
Merge pull request #145 from subCode321/install-openms
jcharkow Mar 4, 2025
64acec6
Merge upstream/main and resolve conflicts, keeping pyopenms_viz==1.0.0
achalbajpai Mar 4, 2025
7771337
add installer to release docker files updated
Mar 4, 2025
76b5d2d
add installer to release docker files updation
Mar 4, 2025
f64a8bb
Merge pull request #148 from achalbajpai/main
jcharkow Mar 4, 2025
9f370f6
Update app name and version display from settings
asterisk-ragavan Mar 5, 2025
c3b82b0
Merge pull request #136 from definitelynotchirag/main
t0mdavid-m Mar 5, 2025
9d816ec
Merge pull request #135 from OpenMS/axelwalter-patch-1
t0mdavid-m Mar 5, 2025
c00fa41
Updated the f string with str() function to display the app name
asterisk-ragavan Mar 5, 2025
49fa2d0
Center text in the UI to display version .
asterisk-ragavan Mar 5, 2025
5c17a4b
Configure_maximum_file_upload_size
ChandraDevs1 Mar 5, 2025
9495ee2
Add configurable file upload size via settings.json and CLI
ChandraDevs1 Mar 5, 2025
e91f7c7
Validate upload size is positive
ChandraDevs1 Mar 5, 2025
f6c383b
added maxUploadSize-value
ChandraDevs1 Mar 6, 2025
01854e0
Add script to run Streamlit app with logging and error handling
ChandraDevs1 Mar 6, 2025
6161dbc
implementated all the comments
ChandraDevs1 Mar 6, 2025
c5c184d
fix the merge issue
ChandraDevs1 Mar 6, 2025
e80292d
Update config.toml
ChandraDevs1 Mar 6, 2025
6b12493
updated the version display style as mentioned.
asterisk-ragavan Mar 6, 2025
21c82ff
remove V from version box
t0mdavid-m Mar 6, 2025
d65c9c9
Merge pull request #150 from asterisk-ragavan/main
t0mdavid-m Mar 6, 2025
7933bc9
maxUploadSize updated
ChandraDevs1 Mar 6, 2025
397e361
Add workspaces directory configuration to settings and update workspa…
definitelynotchirag Mar 6, 2025
f3cd0d8
updated files for adding installer to release
Mar 6, 2025
a6d8ba7
version updated
Mar 6, 2025
3659310
Merge branch 'main' into add-installer-to-release
Srajald Mar 6, 2025
768d258
Update settings.json
Srajald Mar 6, 2025
e8c14e7
Updated the release
Srajald Mar 6, 2025
f459e66
updated doc for local mode
ChandraDevs1 Mar 7, 2025
e7ff3db
apply temp fix for windows ci
t0mdavid-m Mar 7, 2025
2b2661f
checkout file instead of cherry picking
t0mdavid-m Mar 7, 2025
c13685d
fix typo
t0mdavid-m Mar 7, 2025
75f027a
add origin
t0mdavid-m Mar 7, 2025
a181b73
Merge remote-tracking branch 'origin/main' into fix_windows_builds
t0mdavid-m Mar 7, 2025
1c67d5f
also overwrite cipackage
t0mdavid-m Mar 7, 2025
85fc9c3
Merge pull request #161 from t0mdavid-m/fix_windows_builds
t0mdavid-m Mar 7, 2025
c03f4e6
Merge remote-tracking branch 'origin/main' into pr-149
t0mdavid-m Mar 7, 2025
7189314
Update user guide and settings to specify workspace directory in loca…
definitelynotchirag Mar 7, 2025
75fa783
Merge pull request #149 from Srajald/add-installer-to-release
t0mdavid-m Mar 7, 2025
f94ecff
additional teste
Mar 7, 2025
d8bc532
updated timeout
subCode321 Mar 8, 2025
e3857b8
Revert "updated timeout"
subCode321 Mar 8, 2025
15e5a13
updated timeout
subCode321 Mar 8, 2025
7183499
Merge pull request #164 from subCode321/update-timeout
t0mdavid-m Mar 8, 2025
2b6cb9a
test cases added
Mar 10, 2025
7fa851f
test cases improved
Mar 10, 2025
df9aab5
test cases edited
Mar 10, 2025
160ad96
invalid test case added
Mar 10, 2025
a2070ea
more functionality added
Mar 10, 2025
b4209d7
more tests added
Mar 10, 2025
63ad905
Delete testfile.mzML
Srajald Mar 10, 2025
5af2fd7
remove logging
ChandraDevs1 Mar 11, 2025
ed8abcd
Merge branch 'main' into feat/workspace_dir
t0mdavid-m Mar 12, 2025
e3f2fd1
add hint for parent directory
t0mdavid-m Mar 12, 2025
84c6e29
fix linting
t0mdavid-m Mar 12, 2025
c78b10e
Merge pull request #159 from definitelynotchirag/feat/workspace_dir
t0mdavid-m Mar 12, 2025
c3a0b2b
Add CPU visualizer
subCode321 Mar 12, 2025
09ce809
Add cpu vizualizer section
subCode321 Mar 12, 2025
58de157
fix typo
t0mdavid-m Mar 12, 2025
66f1943
move resource utilization section below settings
subCode321 Mar 12, 2025
5c9c8fa
Merge branch 'cpu-visualiser' of github.com-subcode:subCode321/stream…
subCode321 Mar 12, 2025
125871d
test cases added
Mar 12, 2025
1f76990
Merge branch 'additional-tests' of https://github.com/Srajald/streaml…
Mar 12, 2025
94bc3a7
updated
Mar 12, 2025
a509a08
Merge branch 'main' into additional-tests
Srajald Mar 12, 2025
edb7fcc
workspace directory rectified
Mar 12, 2025
a6c7e29
added warning
subCode321 Mar 13, 2025
4e67b8c
fixed minor typo
subCode321 Mar 13, 2025
2b0ef60
Merge pull request #179 from subCode321/fix-button
t0mdavid-m Mar 13, 2025
887157c
Merge pull request #175 from subCode321/cpu-visualiser
t0mdavid-m Mar 13, 2025
cc582b4
Update run_app.py
ChandraDevs1 Mar 13, 2025
1d07663
Merge branch 'main' into Configure-size-upload-limit
ChandraDevs1 Mar 13, 2025
8b4f8b0
Update run_app.py
ChandraDevs1 Mar 13, 2025
d3a80d6
Updated logo reference to use new SVG logo
achalbajpai Mar 13, 2025
8c743c8
Added new OpenMS logo
achalbajpai Mar 13, 2025
3e21099
Updated logo reference in quickstart.py
achalbajpai Mar 13, 2025
1619a14
Merge pull request #170 from achalbajpai/update-openms-logo
jcharkow Mar 13, 2025
aa98c1c
docs: update Windows executable documentation with MajorUpgrade element
achalbajpai Mar 13, 2025
9dc5e33
test cases updated
Mar 14, 2025
2a1a7e5
Merge branch 'main' into additional-tests
Srajald Mar 14, 2025
3e0c71f
feat: automatically load example data in pyOpenMS workflow
achalbajpai Mar 15, 2025
3206617
fix: increase timeout for test launch function test to 30 seconds
achalbajpai Mar 15, 2025
5c8bd14
invalid cases removed
Mar 16, 2025
15a680b
added some code refining
Mar 16, 2025
5fb4b22
Update win_exe_with_embed_py.md
Arslan-Siraj Mar 17, 2025
2d8323c
added doc for packaging with WiX:: Merge pull request #153 from achal…
Arslan-Siraj Mar 17, 2025
30feb90
Clear captcha textbox if user enters wrong captcha
jisnoo123 Mar 17, 2025
c04a45a
Merge pull request #1 from jisnoo123/jisnoo123-patch-1
jisnoo123 Mar 17, 2025
cf5cd82
Add Linux pip install guide
jisnoo123 Mar 17, 2025
d722bb8
easy to read doc
jisnoo123 Mar 17, 2025
cbe9ed9
test: Increase timeout for raw_data_viewer tests and improve formatting
Mar 18, 2025
edbdb16
Delete content/simple_workflow.py
Srajald Mar 18, 2025
b12a3b2
replaced the changes
Mar 18, 2025
9a2d063
Merge branch 'additional-tests' of https://github.com/Srajald/streaml…
Mar 18, 2025
7a9daa7
Placed guidelines for new python env above the existing one
jisnoo123 Mar 18, 2025
1e788c5
Place guide for new Python env above existing
jisnoo123 Mar 18, 2025
a2441d2
add new python env guide above the existing one
jisnoo123 Mar 18, 2025
51a01fd
Merge pull request #186 from achalbajpai/test-timeout-fix
jcharkow Mar 18, 2025
81d599e
Merge pull request #167 from Srajald/additional-tests
jcharkow Mar 18, 2025
0c5fdbb
Merge pull request #156 from ChandraDevs1/Configure-size-upload-limit
jcharkow Mar 19, 2025
78c4178
update : remove example data tab and associated code
achalbajpai Mar 19, 2025
d5756dc
Merge pull request #189 from jisnoo123/main
t0mdavid-m Mar 19, 2025
92a6979
Merge pull request #185 from achalbajpai/upstream-fresh-branch
t0mdavid-m Mar 19, 2025
6a7766a
fix: Prevent Streamlit email prompt on first run by providing credent…
achalbajpai Mar 21, 2025
f5be3ef
Feature: dependabot (#214)
jcharkow Mar 29, 2025
692c6db
Update new logo build icon and page svg (#206)
ketan1406 Mar 29, 2025
b2786f9
fix : OpenMS Executable Release Artifact Upload Issue (#208)
achalbajpai Mar 29, 2025
3352c27
Fixed return type of get_parameters_from_json (#221)
ZenMachina16 Apr 9, 2025
c1c8b9e
Fix/dependabot (#220)
jcharkow Apr 17, 2025
175e027
deps: bump captcha from 0.5.0 to 0.7.1 (#219)
dependabot[bot] Apr 17, 2025
65c152b
deps: bump pyopenms from 3.2.0 to 3.3.0 (#215)
dependabot[bot] Apr 17, 2025
f8a9cb4
Add comprehensive tests for TOPP workflow parameter page (#209)
blenbot Apr 17, 2025
8a11b6d
Merge remote-tracking branch 'template/main' into merge_template
t0mdavid-m May 15, 2025
393b685
remove tests for now
t0mdavid-m May 15, 2025
090518d
add upgrade code
t0mdavid-m May 15, 2025
b2a8fe9
add psutil as requirement
t0mdavid-m May 15, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
version: 2
updates:
- package-ecosystem: "pip"
directory: "/" # Location of your pyproject.toml or requirements.txt
schedule:
interval: "weekly" # Checks for updates every week
commit-message:
prefix: "deps" # Prefix for pull request titles
open-pull-requests-limit: 5 # Limit the number of open PRs at a time
35 changes: 31 additions & 4 deletions .github/workflows/build-windows-executable-app.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@ env:
OPENMS_VERSION: 3.2.0
PYTHON_VERSION: 3.11.0
# Name of the installer
APP_NAME: FLASHApp-0.70
APP_NAME: FLASHApp-0.7.2
APP_UpgradeCode: "69ae44ad-d554-4e3c-8715-7c4daf60f8bb"

jobs:
build-vue-js-component:
Expand Down Expand Up @@ -58,6 +59,17 @@ jobs:
repository: t0mdavid-m/OpenMS
ref: FVdeploy
path: 'OpenMS'

# Temporary fix - until seqan is back online or new OpenMS release (3.4)
- name: Get latest cibuild.cmake
working-directory: OpenMS
run: |
git config user.name "GitHub Actions"
git config user.email "actions@github.com"
git fetch origin develop
git checkout origin/develop -- tools/ci/cibuild.cmake
git checkout origin/develop -- tools/ci/citest.cmake
git checkout origin/develop -- tools/ci/cipackage.cmake

- name: Install Qt
uses: jurplel/install-qt-action@v3
Expand Down Expand Up @@ -213,6 +225,13 @@ jobs:
name: vue-js-dist
path: js-dist

- name: Set Version in settings.json
run: |
$VERSION="${{ github.event.release.tag_name }}"
$content = Get-Content -Raw settings.json | ConvertFrom-Json
$content.version = $VERSION
$content | ConvertTo-Json -Depth 100 | Set-Content settings.json

- name: Download package as artifact
uses: actions/download-artifact@v4
with:
Expand All @@ -227,7 +246,7 @@ jobs:
cp -r openms-package/share ../share

- name: Set up Python (regular distribution)
uses: actions/setup-python@v4
uses: actions/setup-python@v5
with:
python-version: ${{ env.PYTHON_VERSION }} # Use the same version as the embeddable version

Expand Down Expand Up @@ -276,7 +295,15 @@ jobs:

- name: Create .bat file
run: |
echo " start /min .\python-${{ env.PYTHON_VERSION }}\python -m streamlit run app.py local" > ${{ env.APP_NAME }}.bat
echo '@echo off' > ${{ env.APP_NAME }}.bat
echo '' >> ${{ env.APP_NAME }}.bat
echo 'REM Create .streamlit directory in user''s home if it doesn''t exist' >> ${{ env.APP_NAME }}.bat
echo 'if not exist "%USERPROFILE%\.streamlit" mkdir "%USERPROFILE%\.streamlit"' >> ${{ env.APP_NAME }}.bat
echo '' >> ${{ env.APP_NAME }}.bat
echo 'REM Create credentials.toml with empty email to disable email prompt' >> ${{ env.APP_NAME }}.bat
echo 'copy /Y ".streamlit\credentials.toml" "%USERPROFILE%\.streamlit\credentials.toml" > nul' >> ${{ env.APP_NAME }}.bat
echo '' >> ${{ env.APP_NAME }}.bat
echo 'start /min .\python-${{ env.PYTHON_VERSION }}\python -m streamlit run app.py local' >> ${{ env.APP_NAME }}.bat

- name: Create All-in-one executable folder
run: |
Expand Down Expand Up @@ -358,7 +385,7 @@ jobs:
cat <<EOF > streamlit_exe.wxs
<?xml version="1.0"?>
<Wix xmlns="http://schemas.microsoft.com/wix/2006/wi">
<Product Id="8434b940-a943-40f1-ab1f-4db3feedb4ba" Name="${{ env.APP_NAME }}" Language="1033" Version="1.0.0.0" Codepage="1252" Manufacturer="OpenMS Developer Team" UpgradeCode="8d28e8c7-45dc-446c-b889-99a6aea2f1a5">
<Product Id="*" Name="${{ env.APP_NAME }}" Language="1033" Version="1.0.0.0" Codepage="1252" Manufacturer="OpenMS Developer Team" UpgradeCode="${{ env.APP_UpgradeCode }}">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue

Fix static WiX Product version attribute.

The <Product> element’s Version="1.0.0.0" is hardcoded and will diverge from APP_NAME. This can lead to upgrade/patch mismatches. You should inject the actual release version (e.g. from ${{ github.event.release.tag_name }}) into the Version attribute so the installer version tracks the app version.

🤖 Prompt for AI Agents
In .github/workflows/build-windows-executable-app.yaml at line 388, the Product
element's Version attribute is hardcoded to "1.0.0.0". Replace this static
version with a dynamic reference to the actual release version, such as using
the GitHub release tag (e.g., ${{ github.event.release.tag_name }}), so the
installer version matches the app version and avoids upgrade or patch
mismatches.

<Package Id="*" InstallerVersion="300" Compressed="yes" InstallPrivileges="elevated" Platform="x64" />
<Media Id="1" Cabinet="streamlit.cab" EmbedCab="yes" />

Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/test-win-exe-w-embed-py.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ jobs:

env:
PYTHON_VERSION: 3.11.9
APP_UpgradeCode: 4abc2e23-3ba5-40e4-95c9-09e6cb8ecaeb
APP_NAME: OpenMS-StreamlitTemplateApp-Test

steps:
Expand Down Expand Up @@ -115,7 +116,7 @@ jobs:
cat <<EOF > streamlit_exe.wxs
<?xml version="1.0"?>
<Wix xmlns="http://schemas.microsoft.com/wix/2006/wi">
<Product Id="8434b940-a943-40f1-ab1f-4db3feedb4ba" Name="${{ env.APP_NAME }}" Language="1033" Version="1.0.0.0" Codepage="1252" Manufacturer="OpenMS Developer Team" UpgradeCode="8d28e8c7-45dc-446c-b889-99a6aea2f1a5">
<Product Id="*" Name="${{ env.APP_NAME }}" Language="1033" Version="1.0.0.0" Codepage="1252" Manufacturer="OpenMS Developer Team" UpgradeCode="${{ env.APP_UpgradeCode }}">
<Package Id="*" InstallerVersion="300" Compressed="yes" InstallPrivileges="elevated" Platform="x64" />
<Media Id="1" Cabinet="streamlit.cab" EmbedCab="yes" />

Expand Down
4 changes: 4 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ myenv
build.log
run_app.spec
clean-up-workspaces.log
gdpr_consent/node_modules/

# FLASHApp
workspaces
Expand All @@ -23,4 +24,7 @@ workspaces-flashtaggerviewer
# Temp files
**/__pycache__
.DS_Store
*-embed-amd64
*~


8 changes: 5 additions & 3 deletions .streamlit/config.toml
Original file line number Diff line number Diff line change
@@ -1,12 +1,14 @@
[browser]
gatherUsageStats = false

[global]
developmentMode = false

[server]
maxUploadSize = 2000 #MB
port = 8501 # should be same as configured in deployment repo
maxUploadSize = 2000

[theme]

# The preset Streamlit theme that your custom theme inherits from. One of "light" or "dark".
# base =

Expand All @@ -23,4 +25,4 @@ primaryColor = "#29379b"
# textColor =

# Font family for all text in the app, except code blocks. One of "sans serif", "serif", or "monospace".
# font =
# font =
2 changes: 2 additions & 0 deletions .streamlit/credentials.toml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
[general]
email = ""
12 changes: 8 additions & 4 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -73,8 +73,7 @@ RUN wget -q \
RUN mamba --version

# Setup mamba environment.
COPY environment.yml ./environment.yml
RUN mamba env create -f environment.yml
RUN mamba create -n streamlit-env python=3.10
RUN echo "mamba activate streamlit-env" >> ~/.bashrc
SHELL ["/bin/bash", "--rcfile", "~/.bashrc"]
SHELL ["mamba", "run", "-n", "streamlit-env", "/bin/bash", "-c"]
Expand Down Expand Up @@ -115,6 +114,10 @@ RUN make -j4 pyopenms
WORKDIR /openms-build/pyOpenMS
RUN pip install dist/*.whl

# Install other dependencies (excluding pyopenms)
COPY requirements.txt ./requirements.txt
RUN grep -v '^pyopenms' requirements.txt > requirements_cleaned.txt && mv requirements_cleaned.txt requirements.txt
RUN pip install -r requirements.txt

WORKDIR /
RUN mkdir openms
Expand Down Expand Up @@ -172,14 +175,15 @@ RUN mamba run -n streamlit-env python hooks/hook-analytics.py
# Set Online Deployment
RUN jq '.online_deployment = true' settings.json > tmp.json && mv tmp.json settings.json

# Download latest OpenMS App executable for Windows from Github actions workflow.
# Download latest OpenMS App executable as a ZIP file
RUN if [ -n "$GH_TOKEN" ]; then \
echo "GH_TOKEN is set, proceeding to download the release asset..."; \
gh release download --repo ${GITHUB_USER}/${GITHUB_REPO} --pattern "${ASSET_NAME}" --dir /app; \
gh release download -R ${GITHUB_USER}/${GITHUB_REPO} -p "OpenMS-App.zip" -D /app; \
else \
echo "GH_TOKEN is not set, skipping the release asset download."; \
fi


# Run app as container entrypoint.
EXPOSE $PORT
ENTRYPOINT ["/app/entrypoint.sh"]
5 changes: 5 additions & 0 deletions app.py
Original file line number Diff line number Diff line change
@@ -1,8 +1,13 @@
import streamlit as st
from pathlib import Path
import json
# For some reason the windows version only works if this is imported here
import pyopenms

if "settings" not in st.session_state:
with open("settings.json", "r") as f:
st.session_state.settings = json.load(f)

if __name__ == '__main__':
pages = {
"FLASHApp" : [
Expand Down
165 changes: 165 additions & 0 deletions assets/openms_transparent_bg_logo.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
19 changes: 19 additions & 0 deletions docker-compose.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
version: '3.8'

services:
openms-streamlit-template:
build:
context: .
dockerfile: Dockerfile
args:
GITHUB_TOKEN: $GITHUB_TOKEN
image: openms_streamlit_template
Comment on lines +8 to +10
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Verification agent

❓ Verification inconclusive

Build-arg placeholder won’t be expanded – use ${VARNAME} syntax

Compose only substitutes environment variables when they are written as ${VARNAME} (or $$ to escape).
Keeping $GITHUB_TOKEN will pass the literal string to Docker, resulting in an empty value inside the image unless you manually quote & export it again.

-      args:
-        GITHUB_TOKEN: $GITHUB_TOKEN
+      args:
+        GITHUB_TOKEN: ${GITHUB_TOKEN}

(If the variable is optional, use ${GITHUB_TOKEN:-}.)


Use ${VARNAME} for build-arg interpolation in docker-compose.yml

Docker Compose only substitutes variables written as ${VARNAME} (or $$ to escape). Using $GITHUB_TOKEN without braces will pass the literal string "$GITHUB_TOKEN" into the build context, not the value of your environment variable.

Apply this change:

   build:
     context: .
     dockerfile: Dockerfile
     args:
-      GITHUB_TOKEN: $GITHUB_TOKEN
+      GITHUB_TOKEN: ${GITHUB_TOKEN}

If the token may be undefined, you can provide an empty default:

      GITHUB_TOKEN: "${GITHUB_TOKEN:-}"
🤖 Prompt for AI Agents
In docker-compose.yml around lines 8 to 10, the build-arg GITHUB_TOKEN is
referenced as $GITHUB_TOKEN, which will not be expanded by Docker Compose and
will pass the literal string instead. To fix this, change the reference to use
the syntax ${GITHUB_TOKEN} for proper environment variable substitution. If the
variable might be undefined, use the default empty value syntax
${GITHUB_TOKEN:-} to avoid errors.

container_name: openms-streamlit-template
restart: always
ports:
- 8501:8501
volumes:
- workspaces-streamlit-template:/workspaces-streamlit-template
command: streamlit run openms-streamlit-template/app.py
volumes:
workspaces-streamlit-template:
Loading