Skip to content

Add memory_profiler and pympler to CI environment dependencies#768

Merged
charles-turner-1 merged 2 commits intosearch-with-polarsfrom
copilot/add-memory-profiler-ci
Jan 6, 2026
Merged

Add memory_profiler and pympler to CI environment dependencies#768
charles-turner-1 merged 2 commits intosearch-with-polarsfrom
copilot/add-memory-profiler-ci

Conversation

Copy link

Copilot AI commented Jan 6, 2026

Change Summary

PR #767 introduces memory profiling with memory_profiler and pympler imports in production code (intake_esm/_search.py, intake_esm/cat.py). These packages were missing from CI environments, causing import failures.

Added both packages to:

  • ci/environment.yml - main CI build matrix (Python 3.11, 3.12, 3.13)
  • ci/environment-upstream-dev.yml - upstream-dev CI job

Related issue number

Supports PR #767

Checklist

  • Unit tests for the changes exist
  • Tests pass on CI
  • Documentation reflects the changes where applicable
Original prompt

Pull Request: https://github.com/intake/intake-esm/pull/767/files

We need to add memory profiler to the ci environment here to get the tests to run - can you do that for me please


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: charles-turner-1 <52199577+charles-turner-1@users.noreply.github.com>
Copilot AI changed the title [WIP] Add memory profiler to CI environment Add memory_profiler and pympler to CI environment dependencies Jan 6, 2026
Copilot AI requested a review from charles-turner-1 January 6, 2026 05:05
@charles-turner-1 charles-turner-1 marked this pull request as ready for review January 6, 2026 05:20
@charles-turner-1 charles-turner-1 merged commit 8d446ee into search-with-polars Jan 6, 2026
3 of 9 checks passed
@charles-turner-1 charles-turner-1 deleted the copilot/add-memory-profiler-ci branch January 6, 2026 05:28
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