Skip to content

Conversation

@VeriTas-arch
Copy link
Contributor

Contents

  • Trying to unify the doc style with imperative-style instead of a combination of it and descriptive-style. see reference here
  • Minor changes to examples on code format (i.e. add some space to Vec2d variables, for instance, changing from Vec2d(3,4) to Vec2d(3, 4)).
  • Remove a temporary variable (projected_length_times_other_length, line 400). (Such variables may initially exist given the readability of codes, yet seeming unnecessary under some conditions, such as several well-known vector operations).

Summary

Minor changes only.

@viblo viblo merged commit 51735b6 into viblo:master Nov 5, 2024
5 checks passed
@viblo
Copy link
Owner

viblo commented Nov 5, 2024

Thanks for the PR!

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