Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add Tomograms table #988

Merged
merged 17 commits into from
Aug 9, 2024
Merged

feat: Add Tomograms table #988

merged 17 commits into from
Aug 9, 2024

Conversation

bchu1
Copy link
Contributor

@bchu1 bchu1 commented Aug 3, 2024

closes #995

Barebones initial table (UX not finalized yet, waiting until then to do fit & finish). Metadata drawer is empty. Download modal does not target the selected tomogram yet.

Query changes:

  • Added an is_canonical check to the existing tomogram_voxel_spacings field (which we can delete after launch) so that if we need to turn off the flag, the UI will still only display the canonical tomogram.
  • Replaced tomogram_stats count with top-level tomograms_aggregate count that doesn't require FE summing.

BE pieces not supported yet:

  • Distinguish between Portal Standard and Author Submitted tomograms.
  • Deposition date.
  • Alignment ID.

image

@bchu1 bchu1 changed the title Add Tomograms table feat: Add Tomograms table Aug 5, 2024
@bchu1 bchu1 merged commit 420acdd into main Aug 9, 2024
14 checks passed
@bchu1 bchu1 deleted the bchu/table branch August 9, 2024 23:15
github-actions bot added a commit that referenced this pull request Aug 12, 2024
🤖 I have created a release *beep* *boop*
---


##
[1.19.0](web-v1.18.0...web-v1.19.0)
(2024-08-12)


### ✨ Features

* Add Tomograms table
([#988](#988))
([420acdd](420acdd))
* Enable pagination of Annotations table with temporary hacky query
([#992](#992))
([79f7247](79f7247))


### ♻️ Code Refactoring

* download dialog tests
([#1011](#1011))
([f0ece55](f0ece55)),
closes
[#962](#962)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
kne42 added a commit that referenced this pull request Aug 16, 2024
* main: (21 commits)
  feat: add filter panel to single deposition page (#1030)
  chore(main): release web 1.21.0 (#1032)
  feat: Update Tomogram Processing field format and query (#1031)
  chore(main): release web 1.20.0 (#1028)
  feat: Implement collapsing Annotated Objects list (#1024)
  chore: Add e2e test for errors on Neuroglancer site (#1027)
  chore(main): release web 1.19.1 (#1023)
  fix: Fix Neuroglancer URL bug (#1026)
  fix: Dedupe authors (#1018)
  chore(main): release web 1.19.0 (#1019)
  refactor: download dialog tests (#1011)
  feat: Add Tomograms table (#988)
  feat: Enable pagination of Annotations table with temporary hacky query (#992)
  chore(main): release web 1.18.0 (#984)
  feat: Add Tomograms tab to Run page (#983)
  chore: remove extra GHA concurrency lock from prod-deploy.yml (#982)
  feat: add user agent to client requests (#966)
  chore: Add additional test case to TestGetDestinationPath (#955)
  chore(main): release web 1.17.0 (#971)
  ci: update frontend release pr filter (#977)
  ...
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.

Multiple tomogram table (structure not column content)
2 participants