Skip to content

[CI/CLEANUP] Remove caldip code and the dependabot pip bumps#26

Merged
eleanorfrajka merged 5 commits into
mainfrom
caldips
Jul 5, 2026
Merged

[CI/CLEANUP] Remove caldip code and the dependabot pip bumps#26
eleanorfrajka merged 5 commits into
mainfrom
caldips

Conversation

@eleanorfrajka

@eleanorfrajka eleanorfrajka commented Jul 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR appears to clean up the repository around “caldip”/dependency housekeeping while also improving packaging and documentation/notebook formatting.

Changes:

  • Updated requirements.txt extras by removing ctd_tools and adding seasenselib.
  • Added oceanarray/__init__.py to formalize the oceanarray package interface via __all__.
  • Reformatted Jupyter notebooks’ JSON (string → list form) and updated documentation, including adding a new caldips doc page.

Reviewed changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
requirements.txt Adjusts pip/dev-only extra dependency list.
oceanarray/init.py Adds package initializer and defines __all__.
notebooks/demo_step1.ipynb Notebook formatting updates and adds a cell that displays combined_ds.
notebooks/demo_check_clock.ipynb Notebook formatting updates; comments out _use toggle line.
docs/source/data_format_specification.rst Reworks/labels spec as “old” and updates tables/wording.
docs/source/calibration_dips.rst Adds documentation page describing calibration dips procedure.
.gitignore Ignores caldip prototype scripts/data artifacts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread oceanarray/__init__.py
Comment thread docs/source/calibration_dips.rst Outdated
Comment thread docs/source/calibration_dips.rst Outdated
Comment thread docs/source/calibration_dips.rst Outdated
Comment thread notebooks/demo_step1.ipynb
@eleanorfrajka eleanorfrajka merged commit c4d5698 into main Jul 5, 2026
4 checks passed
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