Skip to content

Drop explicit TensorBoard dependency#12

Merged
kmaziarz merged 3 commits intomainfrom
kmaziarz/drop-tensorboard
Mar 31, 2026
Merged

Drop explicit TensorBoard dependency#12
kmaziarz merged 3 commits intomainfrom
kmaziarz/drop-tensorboard

Conversation

@kmaziarz
Copy link
Copy Markdown
Contributor

During integration with syntheseus, I found a conflict between the current dependency pins in RetroChimera and those of the MEGAN model: we have a strict pin of tensorboard==2.12.0, which is not compatible with tensorflow==2.20.0 pinned in MEGAN. However, RetroChimera itself doesn't use TensorBoard directly anymore: we used to use it for logging, but have since switched to Weights & Biases. It is still an indirect dependency through syntheseus-root-aligned -> OpenNMT-py -> tensorboard>=2.3; after dropping the strict pin in pyproject.toml, the dependency resolver can choose whatever works for MEGAN and OpenNMT (in this case, the newest 2.20.0 is fine).

Also, I raise the minimum version of syntheseus to 0.7.2, to pull in a recent checkpoint downloading bugfix from 0.7.2.

@kmaziarz kmaziarz requested review from jla-gardner and mrwnmsr March 23, 2026 13:37
@kmaziarz kmaziarz changed the title Drop TensorBoard dependency Drop explicit TensorBoard dependency Mar 23, 2026
kmaziarz added a commit that referenced this pull request Mar 31, 2026
@kmaziarz kmaziarz force-pushed the kmaziarz/drop-tensorboard branch from d7a3b5a to b8a54de Compare March 31, 2026 15:53
@kmaziarz kmaziarz enabled auto-merge (squash) March 31, 2026 15:54
@kmaziarz kmaziarz merged commit 257628e into main Mar 31, 2026
6 checks passed
@kmaziarz kmaziarz deleted the kmaziarz/drop-tensorboard branch March 31, 2026 16:06
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