Bradley Solliday

Results 9 comments of Bradley Solliday

Huh. I didn't realize that was possible. Well, I really only cared about gtsam. If we have time to kill, we could get this merged (perhaps by making it easier...

If you don't update pip, the CI gives this error while installing `pip install build/lcmtypes/python2.7` ``` Installing collected packages: lcmtypes Successfully installed lcmtypes-0.5.0 --- Logging error --- Traceback (most recent...

To summarize the state of this issue, issue 1 has been fixed for some time now, and issue 2 was resolved recently (here: https://github.com/symforce-org/symforce/pull/264). Aaron's hypothesis that this had to...

CI is passing without this, so evidently it's no longer necessary.

Reviews in this chain: └**https://github.com/symforce-org/symforce/pull/295** **Simplify geo_factors_codegen w/ skip_dir_nesting**  └https://github.com/symforce-org/symforce/pull/294 Add diagonal & isotropic sqrt_infos in geo factors

| # | head | base | diff | date | summary | | - | - | - | - | - | - | | 0 | [f9a6f2d0](/symforce-org/symforce/commit/f9a6f2d0407d955b455af378f0d4e7546d23a3b8)...

Reviews in this chain: └https://github.com/symforce-org/symforce/pull/295 Simplify geo_factors_codegen w/ skip_dir_nesting  └**https://github.com/symforce-org/symforce/pull/294** **Add diagonal & isotropic sqrt_infos in geo factors**

| # | head | base | diff | date | summary | | - | - | - | - | - | - | | 0 | [50c4a3af](/symforce-org/symforce/commit/50c4a3af2046f537e6b2ddc90ca9606354919ca1)...

> I see the mention that `sym.gen` would require the changes in this PR regardless, are we planning to look into that next/later? I kind of assumed we would. Though,...