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

Revise instance definitions in Ref and RPI topologies #1473

Merged
merged 14 commits into from
May 24, 2022

Conversation

bocchino
Copy link
Collaborator

  • Update the FPP version
  • Specify the implementation type in the model where appropriate
  • Use new extern declarations in generated C++ code to simplify handwritten C++

Incorporates changes in #1466. Enables #1460.

bocchino added 9 commits May 18, 2022 15:42
Update FPP version
Use new FPP syntax for concrete instance types
Use new FPP syntax for instance types
Use new FPP syntax for LinuxTime instance
Remove special C++ code for blockDriver
Remove specialized code for declaring linuxTimer instance
@LeStarch
Copy link
Collaborator

Looks fine to me.

@LeStarch LeStarch merged commit 63bebd2 into nasa:devel May 24, 2022
danjwait added a commit to danjwait/2022_fprime that referenced this pull request May 26, 2022
LeStarch added a commit that referenced this pull request Jun 29, 2022
* Revise Ref topology model

Update FPP version
Use new FPP syntax for concrete instance types

* Revise RPI example

Use new FPP syntax for instance types

* Revise Ref and RPI topologies

Use new FPP syntax for LinuxTime instance

* Update FPP version

* Update FPP version

* Revise Ref topology

Remove special C++ code for blockDriver

* Update RPI topology

Remove specialized code for declaring linuxTimer instance

* Update FPP version

* lestarch: bumping latest version of FPP

* Update .lgtm.yml

* lestarch: installing sbt

* lestarch: adding pip upgrade

Co-authored-by: M Starch <LeStarch@googlemail.com>
@bocchino bocchino deleted the extern-decls branch July 28, 2022 00:46
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