borglab / borglab/SwiftFusion

”scalar“ is a bad name for a variable or parameter

Open
#131 0 comments 0 reactions 1 assignee Claimed by @marcrasi View on GitHub
Dominant language
Swift
Stars
118
Forks
14
PR merge metrics
No merged PRs in 30d

Description

Describe the role, not the type! e.g.

```swift
/// Returns Jacobians that scale each element by `scalar`.
func jacobians(scalar: Double) -> AnyGaussianFactorArrayBuffer {
```

should be:

```swift
/// Returns Jacobians that scale each element by `scaleFactor`.
func jacobians(scaleFactor: Double) -> AnyGaussianFactorArrayBuffer {
```

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.