Add RawFrame to autogen_headers

[mcm001]

Companion to wpilibsuite/allwpilib#7609

[mcm001]

Why is wpi__RawFrame.hpp trying to include rpygen/wpi__WPI_RawFrame.hpp? I see ./subprojects/robotpy-wpiutil/wpiutil/rpy-include/rpygen/wpi__RawFrame.hpp locally. but that doesn't match the name. Where's the WPI_ prefix coming from?

[auscompgeek]

It's seeing the WPI_RawFrame base class and generating an include for the base class' trampoline.

[mcm001]

Oh interesting. Do I have any way around that? I could just add manual pybind code to the module definition in main.cpp, since it looks like that's true even if I try to add inline code? Or what should I do?

[auscompgeek]

You could try adding force_no_trampoline: true to the YAML for that class. But looking at the YAML the build spits out in its warning I'm not entirely sure that'll work. That's why I suggested just ignoring all the items we don't need.

[mcm001]

How would I get robotpy to ignore everything except for the WPI_TimestampSource enum? Can I do that in the class' YAML?

[mcm001]

Looks like spam ignore: true?

[auscompgeek]

Oh right, just remembered we have ignored_bases if we do want to expose the RawFrame class.

[mcm001]

I don't think it makes sense to quote yet. RawFrame is a thin wrapper around a C array of image data - i'm not sure there's a great way (or even a reason yet?) to wrap it in a numpy array for use with opencv stuff. Might be cool to think about in the future though!

Either way, if this is merged, CI on wpilibsuite/allwpilib#7609 should become green :)

[virtuald]

Looks like spam ignore: true?

That's fine, but in the future there's also defaults.ignore:

defaults:
  ignore: true

[auscompgeek]

Guessing not worth exposing this enum to Python, since there's an equivalent C++ enum in cscore?

[mcm001]

As I read it yes, the cscore side enum is what the rest of cscore uses in its public api, not the RawFrame version.

[auscompgeek]

LGTM