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

open PR to get tag-aligner alab article into staging #685

Merged
merged 11 commits into from
Jun 5, 2024
Merged

Conversation

rennis250
Copy link
Contributor

No description provided.

@rennis250 rennis250 requested a review from N-M-T May 14, 2024 14:11
alpha-lab/tag-aligner/index.md Show resolved Hide resolved

![Depiction of the sparse 3D model produced by our Reference Image Mapper](./rim_3d_model.png)

The white dots on this image (statue of [Theodor Koch-Grünberg](https://en.wikipedia.org/wiki/Theodor_Koch-Grunberg)) represent key points of a sparse 3D model built from a RIM enrichment scanning recording. The model is used by RIM to calculate scene camera positions in an arbitrary coordinate system.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't this be a caption of the image? Can you make it a caption?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This requires using a markdown plugin and some custom code. It seemed more effort than it is worth:

vuejs/vitepress#892

Or rather, I did not want to break anything significant about the pupil-docs system.

alpha-lab/tag-aligner/index.md Outdated Show resolved Hide resolved
alpha-lab/tag-aligner/index.md Outdated Show resolved Hide resolved
@N-M-T N-M-T merged commit dc21c75 into master Jun 5, 2024
8 checks passed
@N-M-T N-M-T deleted the tag_aligner_alab branch June 6, 2024 00:23
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.

2 participants