Skip to content

[BEP032] Moved coord system related entities to correct yaml #2058

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 2 commits into
base: bep032
Choose a base branch
from
Draft

[BEP032] Moved coord system related entities to correct yaml #2058

wants to merge 2 commits into from

Conversation

ree-gupta
Copy link
Member

@ree-gupta ree-gupta commented Feb 19, 2025 •
edited
Loading

  • Add enums for MicroephysCoordSys - similar to what is used already here
  • Add addendum for user in IntendedFor

@ree-gupta ree-gupta requested a review from yarikoptic February 19, 2025 14:54
@yarikoptic
Copy link
Collaborator

I see TODOs yet to be addressed, so I will mark it as "draft" for now . Please mark "ready for review" when ready

@yarikoptic yarikoptic marked this pull request as draft March 19, 2025 15:07
@robertoostenveld
Copy link
Collaborator

for reference (wrt the meeting just now): the coordinate system enums would be a subset of https://openminds-documentation.readthedocs.io/en/latest/instance_libraries/commonCoordinateSpaceVersions.html, corresponding to the non-human ones

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yarikoptic yarikoptic Awaiting requested review from yarikoptic

@erdalkaraca erdalkaraca Awaiting requested review from erdalkaraca erdalkaraca is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ree-gupta @yarikoptic @robertoostenveld