Rename SensorTSLM to TimeCap#71
Closed
max-rosenblattl wants to merge 1 commit intomainfrom
Closed
Conversation
Contributor
📝 WalkthroughWalkthroughThis PR comprehensively rebrand the project from "SensorTSLM" to "TimeCap" by updating SPDX copyright headers across 70+ files, along with documentation, configuration paths, CLI descriptions, and docstrings. No functional code or APIs are changed. ChangesProject Rebranding: SensorTSLM → TimeCap
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
⚔️ Resolve merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
exporters/lean.py (1)
37-37:⚠️ Potential issue | 🟡 Minor | ⚡ Quick winUpdate metadata key to reflect project rename.
The schema metadata key was not updated during the SensorTSLM → TimeCap rename. The key
"sensortslm.schema_version"should be changed to"timecap.schema_version"(or similar, consistent with the new project identity) to match the comprehensive rename documented in commit d259efe.While no code readers depend on this metadata key internally, the naming inconsistency creates a mismatch between the project identity (TimeCap) and the schema metadata it writes.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@exporters/lean.py` at line 37, Update the metadata key used for schema version so it matches the project rename from SensorTSLM to TimeCap: replace the string key "sensortslm.schema_version" with "timecap.schema_version" wherever it’s set (e.g., the dict literal in exporters/lean.py that assigns SCHEMA_VERSION) so the exported metadata uses the new project identity consistently.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@annotator.py`:
- Line 3: Replace the malformed SPDX line that currently reads
"SPDX-FileCopyrightText: 2026 This source file is part of the TimeCap
open-source project." with a normal Python comment containing the
project/branding text, and if an SPDX copyright notice is required add a proper
SPDX field only for copyright (e.g., SPDX-FileCopyrightText with an actual
copyright holder/year) — update the annotator.py header to remove branding from
the SPDX token and move it to a plain comment while ensuring any real SPDX
metadata fields contain only valid copyright statements.
In `@synthesizers/locomotion.py`:
- Line 3: Replace the SPDX-style attribution comment "SPDX-FileCopyrightText:
2026 This source file is part of the TimeCap open-source project." with a plain
comment (no SPDX tag) containing the project attribution text so SPDX tooling
won't mis-parse it; locate the top-of-file comment in synthesizers/locomotion.py
and change the SPDX-FileCopyrightText token to a normal comment line (e.g.,
start with # and remove "SPDX-"), leaving any real SPDX metadata tags untouched
elsewhere.
---
Outside diff comments:
In `@exporters/lean.py`:
- Line 37: Update the metadata key used for schema version so it matches the
project rename from SensorTSLM to TimeCap: replace the string key
"sensortslm.schema_version" with "timecap.schema_version" wherever it’s set
(e.g., the dict literal in exporters/lean.py that assigns SCHEMA_VERSION) so the
exported metadata uses the new project identity consistently.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 1851c7a3-9104-42e1-a83b-225886821bae
📒 Files selected for processing (70)
.github/workflows/static-analysis.ymlCONTRIBUTORS.mdREADME.mdaggregators.pyannotator.pycaptionizer.pycurriculum_learning.pydetectors/__init__.pydetectors/spike.pydetectors/trend.pyevaluation/README.mdevaluation/__init__.pyevaluation/evaluator.pyevaluation/io/__init__.pyevaluation/io/writer.pyevaluation/metrics/__init__.pyevaluation/metrics/classification.pyevaluation/run_eval.pyevaluation/tasks/__init__.pyevaluation/tasks/base.pyevaluation/tasks/metabolic.pyexplorer.pyexporters/lean.pyextractors/__init__.pyextractors/cross_channel.pyextractors/generative.pyextractors/semantic.pyextractors/statistical.pyextractors/structural.pymhc/__init__.pymhc/constants.pymhc/cross_channel.pymhc/dataset.pymhc/transformer.pymhc_weekly/__init__.pymhc_weekly/constants.pymhc_weekly/dataset.pymhc_weekly/transformer.pymodels/__init__.pymodels/base.pymodels/client.pymodels/local.pymodels/opentslm_native.pyreviewer.pyscripts/export_captions.pyscripts/export_captions_sharded.shscripts/metabolic_finetune.pyscripts/sherlock_setup.shsynthesizers/__init__.pysynthesizers/_base.pysynthesizers/_helper.pysynthesizers/_workout.pysynthesizers/cardio.pysynthesizers/locomotion.pysynthesizers/mind_body.pysynthesizers/other_activity.pysynthesizers/sleep.pysynthesizers/stationary_activity.pytime_series_datasets/__init__.pytime_series_datasets/joint_qa_dataset.pytime_series_datasets/mhc_base_qa_dataset.pytime_series_datasets/mhc_caption_qa_dataset.pytime_series_datasets/mhc_label_lookup.pytime_series_datasets/mhc_metabolic_qa_dataset.pytime_series_datasets/mhc_multi_label_qa_dataset.pytimef/__init__.pytimef/schema.pytransformer.pyutil.pyvisualizer.py
| # | ||
| # SPDX-FileCopyrightText: 2026 Stanford University, ETH Zurich, and the project authors (see CONTRIBUTORS.md) | ||
| # SPDX-FileCopyrightText: 2026 This source file is part of the SensorTSLM open-source project. | ||
| # SPDX-FileCopyrightText: 2026 This source file is part of the TimeCap open-source project. |
Contributor
There was a problem hiding this comment.
Use a normal comment here; this is a malformed SPDX copyright field.
Line 3 currently encodes project-branding text as SPDX-FileCopyrightText, which is not a copyright notice and may fail SPDX/license scanning. Keep SPDX fields for actual copyright statements and move branding text to a plain comment.
Suggested fix
-# SPDX-FileCopyrightText: 2026 This source file is part of the TimeCap open-source project.
+# This source file is part of the TimeCap open-source project.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| # SPDX-FileCopyrightText: 2026 This source file is part of the TimeCap open-source project. | |
| # This source file is part of the TimeCap open-source project. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@annotator.py` at line 3, Replace the malformed SPDX line that currently reads
"SPDX-FileCopyrightText: 2026 This source file is part of the TimeCap
open-source project." with a normal Python comment containing the
project/branding text, and if an SPDX copyright notice is required add a proper
SPDX field only for copyright (e.g., SPDX-FileCopyrightText with an actual
copyright holder/year) — update the annotator.py header to remove branding from
the SPDX token and move it to a plain comment while ensuring any real SPDX
metadata fields contain only valid copyright statements.
| # | ||
| # SPDX-FileCopyrightText: 2026 Stanford University, ETH Zurich, and the project authors (see CONTRIBUTORS.md) | ||
| # SPDX-FileCopyrightText: 2026 This source file is part of the SensorTSLM open-source project. | ||
| # SPDX-FileCopyrightText: 2026 This source file is part of the TimeCap open-source project. |
Contributor
There was a problem hiding this comment.
Use a non-SPDX comment for project attribution text
Line 3 uses SPDX-FileCopyrightText for a project-branding sentence, which is not a copyright notice and may break SPDX/compliance tooling parsing. Keep SPDX tags only for SPDX-defined metadata.
Proposed fix
-# SPDX-FileCopyrightText: 2026 This source file is part of the TimeCap open-source project.
+# This source file is part of the TimeCap open-source project.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| # SPDX-FileCopyrightText: 2026 This source file is part of the TimeCap open-source project. | |
| # This source file is part of the TimeCap open-source project. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@synthesizers/locomotion.py` at line 3, Replace the SPDX-style attribution
comment "SPDX-FileCopyrightText: 2026 This source file is part of the TimeCap
open-source project." with a plain comment (no SPDX tag) containing the project
attribution text so SPDX tooling won't mis-parse it; locate the top-of-file
comment in synthesizers/locomotion.py and change the SPDX-FileCopyrightText
token to a normal comment line (e.g., start with # and remove "SPDX-"), leaving
any real SPDX metadata tags untouched elsewhere.
Collaborator
Author
|
Superseded — recreating off latest main for a cleaner diff. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Current situation
Rebrand from
SensorTSLMtoTimeCap. Pure text rename — no code logic, file paths, or imports change. Touches SPDX headers, README, docstrings, argparse descriptions,\$HOME/SensorTSLMdefaults in helper scripts, and the~/SensorTSLMreferences inevaluation/README.md.Release Notes
SensorTSLMtoTimeCap.Documentation
README.md,evaluation/README.md, and SPDX headers updated to useTimeCap. No new docs.Testing
git grep SensorTSLMreturns no matches in tracked files after the rename.Code of Conduct & Contributing Guidelines