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

Merge upstream changes #135

Closed
wants to merge 65 commits into from
Closed

Conversation

patkenneally
Copy link
Collaborator

  • Tickets addressed: NA
  • Review: By commit
  • Merge strategy: Merge (no squash)

Description

git fetch upstream
while on origin/develop
git checkout -b feature/merge-upstream-changes
git merge upstream/develop
fix and mark as resolved any merge conflicts then
git merge --continue
then make PR against origin/develop with branch merge-upstream-changes

Verification

Test suit must run as expected (pass)

Documentation

None

Future work

None

schaubh and others added 30 commits June 23, 2023 08:56
Addition of pinholeCamera.rst file and illustrative image pinholeCamera.png
Added release notes.
Addition of test_unitPinholeCamera.py
I needed a function to convert a Eigen::MatrixXi to a C array for the pixel argument in pinholeCamera.cpp for the LandmarkMsgPayload.h message

I added a new feature in gravCoeffOps.py to extend reading capabilites of polyhedron shape models
Addition of an integrated example script to show how to use pinholeCamera module. It adds: scenarioSmallBodyLandmarks.py and eros007790.tab to generate landmarks distribution based on polyhedron shape. It modifies test_scenarioTest.py to run the integrated example script with pytest
I moved the new message definition commit to the pinhole camera commit with cpp/h files as that commit would not compile without that message definition.  Thanks for contributing this code!
schaubh and others added 28 commits August 18, 2023 12:08
Previously, calling setRNGSeed(this->RNGSeed) in the constructor means that this->RNGSeed still had the default value and could not be set from Python
Previously, unlike what is written in the comment at line 126, the noise model was disabled if all three components of this->senNoiseStd are uninitialized. Now the code already disables the noise model if only one of the 3 components of this->senNoiseStd is uninitialized
…lization

Magnetometer fix noise initialization
Make C-module instantiation the same as C++ instantiation #316
…awFlag

Refactor/418 mrp feedback control law flag
@patkenneally patkenneally deleted the feature/merge-upstream-changes branch August 31, 2023 22:09
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.

6 participants