copilot icon indicating copy to clipboard operation
copilot copied to clipboard

Add ability to add names / labels to streams

Open fdedden opened this issue 5 years ago • 2 comments

Being able to give names or labels to streams has the following benefits:

  • Traceability between the original Copilot specification and C code is improved greatly, which can be a really important point for certification purposes.
  • The generated C code will be easier to read, again helping with certification.
  • Same approach can possibly used for name arguments of triggers, making the generated header file readable for people that don't know about the Copilot specification.

There are some existing pull requests by @avieth related to this enhancement: https://github.com/Copilot-Language/copilot-core/pull/18 https://github.com/Copilot-Language/copilot-language/pull/9 https://github.com/Copilot-Language/copilot-c99/pull/27

These are denied and closed for now, as they implement a way which we deem to specific. They might come in handy later on.

fdedden avatar Apr 07 '20 19:04 fdedden

I don't know if this is the right place to drop this idea, but I was thinking the other day that it might work out nicely to use the magic HasCallStack constraint to implement some form of traceability for stream and trigger definitions, etc. CF: https://hackage.haskell.org/package/base-4.16.0.0/docs/GHC-Stack.html#t:HasCallStack

Basically, you can add the HasCallStack constraint to any Haskell declaration, and the compiler with automatically populate it with information about call sites. That information is propagated to error calls and thrown exceptions, but can also be directly fetched using the callStack combinator. Adding HasCallStack annotations into the user-facing APIs for copilot, and then grabbing those stacks and stashing them into high-level stream definitions would allow a pretty lightweight way to maintain traceability connections to specification code.

robdockins avatar Oct 29 '21 21:10 robdockins

Tagging @RyanGlScott for awareness.

ivanperez-keera avatar Mar 06 '24 22:03 ivanperez-keera