Skip to content

release: bump version to 1.1 and summarize structural cleanup#50

Merged
melvinbarbaux merged 7 commits intomainfrom
release-1.1-structural-cleanup
Apr 3, 2026
Merged

release: bump version to 1.1 and summarize structural cleanup#50
melvinbarbaux merged 7 commits intomainfrom
release-1.1-structural-cleanup

Conversation

@melvinbarbaux
Copy link
Copy Markdown
Collaborator

@melvinbarbaux melvinbarbaux commented Mar 26, 2026

Summary

What does this PR change?

Checklist

  • Tests added or updated
  • CI green
  • Changelog updated
  • Documentation updated

Notes

Anything reviewers should know?

@melvinbarbaux melvinbarbaux self-assigned this Mar 26, 2026
@melvinbarbaux melvinbarbaux added the enhancement New feature or request label Mar 26, 2026
@chatgpt-codex-connector
Copy link
Copy Markdown

chatgpt-codex-connector bot commented Mar 26, 2026

💡 Codex Review

ModSSC/src/modssc/preprocess/models_backends/open_clip.py

Line 8 in 6c7490e

from modssc.cache.model import resolve_openclip_cache_dir

P1 Badge Restore model cache module for backend imports

This change removes src/modssc/model_cache.py but rewires backend modules to import modssc.cache.model (including this file plus sentence_transformers.py, torchaudio_wav2vec2.py, image_pretrained.py, and audio_pretrained.py) without adding a new src/modssc/cache/model.py. In environments where these backends are imported, Python raises ModuleNotFoundError: No module named 'modssc.cache', so preprocessing/supervised backends fail before any optional-dependency guard can run.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@melvinbarbaux melvinbarbaux merged commit 6c6c4cc into main Apr 3, 2026
11 checks passed
@melvinbarbaux melvinbarbaux deleted the release-1.1-structural-cleanup branch April 3, 2026 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@melvinbarbaux melvinbarbaux

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@melvinbarbaux