Implement array hashing in merge, as done in concatenate

Open
#7,241 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
55/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Quiet
Tech stack
python
Domain
performance

Research direction

Start by comparing how merge handles arrays with the existing array-hashing behavior in concatenate, focusing on large and lazy aux-coords. Read the linked example from issue 7063 to understand the slow case. Done means merge uses equivalent array hashing and avoids the reported performance problem.

Written by the indexing model from the issue text.

Description

Without this, merge can be very slow with large aux-coords, and especially with lazy ones
See example

Dominant language
Python
Stars
724
Forks
317
Avg merge
3d 20h
Merged PRs (30d)
9

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from SciTools/iris

All issues in SciTools/iris

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.