Skip to content

Add waterlog app interface#246

Merged
bruno-f-cruz merged 7 commits into
mainfrom
feat-waterlog-ui
May 15, 2026
Merged

Add waterlog app interface#246
bruno-f-cruz merged 7 commits into
mainfrom
feat-waterlog-ui

Conversation

@bruno-f-cruz
Copy link
Copy Markdown
Member

@bruno-f-cruz bruno-f-cruz commented May 14, 2026

Adds WaterlogApp, an ExecutableApp for launching the AIBS MPE waterlog executable with settings-driven CLI arguments.

Details:

  • WaterlogSettings (pydantic-settings) — captures username, mouse ID, weight, water earned/supplement, comment
  • WaterlogApp.run() uses LocalDetachedExecutor (see Add LocalDetachedExecutor #249) — waterlog is launched fire-and-forget since the caller doesn't need to wait for it
  • Falls back to shutil.which("waterlog.exe") if the default %PROGRAMFILES%\AIBS_MPE\waterlog\waterlog.exe is not found

Depends on #249 — rebase/merge that first.

@bruno-f-cruz
Copy link
Copy Markdown
Member Author

@micahwoodard @patricklatimer This is the pattern I had in mind when using CLI apps with executors. Never got around to formalizing it, but in case you are interested. These BaseSettings could obviously be defined elsewhere and imported.

Comment thread src/clabe/aind_apps/waterlog.py Outdated
Comment thread src/clabe/aind_apps/waterlog.py Outdated
Comment thread src/clabe/aind_apps/waterlog.py
@bruno-f-cruz bruno-f-cruz merged commit 923e645 into main May 15, 2026
9 checks passed
@bruno-f-cruz bruno-f-cruz deleted the feat-waterlog-ui branch June 1, 2026 19:42
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