GroupManifold accepts two arguments

[olivierverdier]

The default representation for GroupManifold is LeftInvariantRepresentation, this ensures backward compatibility with previous versions of Manifolds.jl

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.69%. Comparing base (11f0a6c) to head (58698db).
Report is 2 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #747   +/-   ##
=======================================
  Coverage   99.69%   99.69%           
=======================================
  Files         123      123           
  Lines       11415    11418    +3     
=======================================
+ Hits        11380    11383    +3     
  Misses         35       35           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kellertuer]

Looks reasonable to me. Thanks :) Could you wait a bit for #748, then add this as an entry to the changelog (including setting the date); and bump the version in the project toml? Then wo could register this as 0.10.2

[mateuszbaran]

I think this is fine, I'd only adding this default value to the docstring of GroupManifold and changelog.

[kellertuer]

lgtm.