[ENH] add Table table and endpoint #1219
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| analyses, points, tables = process_coordinates(pmid, s, metadata_row) | ||
| to_commit.extend(points) | ||
| to_commit.extend(analyses) | ||
| to_commit.extend(tables) |
There was a problem hiding this comment.
In ace_ingestion_logic, the call analyses, points, tables = process_coordinates(pmid, s, metadata_row) now expects three return values, but process_coordinates still returns only two when a study has no coordinates (the KeyError branch at lines 574-578). If an ACE metadata row is missing from coordinates_df this unpack will raise ValueError and abort ingestion instead of skipping the study as before. Update the error path to return a third value so missing-coordinate records no longer crash the ingest.
Useful? React with 👍 / 👎.
closes #1207