-
Notifications
You must be signed in to change notification settings - Fork 9
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 portal standard badge to tomograms table and integrate alignment metadata accordion with new API #1179
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks @bchu1 !
values: [ | ||
parseAffineTransformationMatrix( | ||
alignment?.affineTransformationMatrix, | ||
) ?? '1 0 0 0 0 1 0 0 0 0 1 0 0 0 0 1', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where does the default matrix come from? Maybe a comment would be helpful.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's just the identity matrix, I extracted it into a variable.
ad32116
to
6650686
Compare
* main: docs: Fix docs site FAQ display and linking (#1189) fix: Stop querying depositionTitle (#1187) docs: Fix all links to be internal (#1175) feat: Add portal standard badge to tomograms table and integrate alignment metadata accordion with new API (#1179) feat: Integrate more V2 API queries (#1174)
🤖 I have created a release *beep* *boop* --- ## [1.31.0](web-v1.30.0...web-v1.31.0) (2024-10-03) ### ✨ Features * Add portal standard badge to tomograms table and integrate alignment metadata accordion with new API ([#1179](#1179)) ([1de70a1](1de70a1)) * Add standard tomogram badge to tomogram selector ([#1181](#1181)) ([ac4ad27](ac4ad27)) * Add tooltip to Alignment ID field in tomogram sidebar ([#1196](#1196)) ([fad5207](fad5207)) * Integrate more V2 API queries ([#1174](#1174)) ([92f80a7](92f80a7)) * Migrate and start using tomograms query in UI ([#1169](#1169)) ([d21ffd9](d21ffd9)) * Undo changes for transforming annotations and add new callout for mismatched alignments ([#1191](#1191)) ([ea1598d](ea1598d)) * update annotation row name ([#1163](#1163)) ([6164201](6164201)) * Update to generic annotation transformation callout during download ([#1186](#1186)) ([5597524](5597524)) ### 🐞 Bug Fixes * address misc UI/UX bugs ([#1192](#1192)) ([178737d](178737d)) * change incorrect deposition links ([#1133](#1133)) ([dc642e5](dc642e5)) * Stop querying depositionTitle ([#1187](#1187)) ([07a08d4](07a08d4)) ### 🧹 Miscellaneous Chores * prefix IDs across the portal ([#1178](#1178)) ([1732fb2](1732fb2)) --- 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>
#1097
Learn more link not available yet.
Canonical alignment not available yet.