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

Document that we use row-vectors for transform math #330

Merged
merged 1 commit into from Mar 28, 2019

Conversation

@Manishearth
Copy link
Member

Manishearth commented Mar 28, 2019

fixes #329

r? @nical @pcwalton


This change is Reviewable

@Manishearth
Copy link
Member Author

Manishearth commented Mar 28, 2019

I would like to switch us over to column-major column-vector notation, but that doesn't have to happen just yet.

@nical
Copy link
Collaborator

nical commented Mar 28, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Mar 28, 2019

📌 Commit da0f5b1 has been approved by nical

@bors-servo
Copy link
Contributor

bors-servo commented Mar 28, 2019

Testing commit da0f5b1 with merge acc2b07...

bors-servo added a commit that referenced this pull request Mar 28, 2019
Document that we use row-vectors for transform math

fixes #329

r? @nical @pcwalton

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/euclid/330)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Mar 28, 2019

☀️ Test successful - checks-travis
Approved by: nical
Pushing acc2b07 to master...

@bors-servo bors-servo merged commit da0f5b1 into servo:master Mar 28, 2019
2 of 3 checks passed
2 of 3 checks passed
Travis CI - Pull Request Build Created
Details
Travis CI - Pull Request Build Passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants
You can’t perform that action at this time.