Skip to content

skip obs that error#44

Open
dannda wants to merge 1 commit intomainfrom
fix/anndata-obs
Open

skip obs that error#44
dannda wants to merge 1 commit intomainfrom
fix/anndata-obs

Conversation

@dannda
Copy link
Collaborator

@dannda dannda commented Nov 4, 2025

Description

catch and skip obs that error

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature (non-breaking change which adds functionality)
  • Optimization (non-breaking, back-end change that speeds up the code)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Checklist

  • All tests pass (eg. npm test)
  • Pre-commit hooks run successfully (eg. pre-commit run --all-files)

@will-moore
Copy link
Contributor

The code changes look good, but I don't know how to test. Is there some sample data somewhere that I could use?

Copy link
Collaborator

@davehorsfall davehorsfall left a comment

Choose a reason for hiding this comment

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

LGTM

@will-moore
Copy link
Contributor

@davehorsfall Thanks for the links. Unfortunately they are both giving me a bunch of errors like:

Viewer.jsx:102 Uncaught TypeError: Cannot read properties of undefined (reading '0')
    at Viewer.jsx:102:35
    at Array.map (<anonymous>)
    at Viewer.jsx:89:8
    at updateMemo (react-dom.development.js:16427:19)

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.

3 participants

Comments