-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
initial "experimental" boards #14013
Conversation
cf0964c
to
4695145
Compare
Nice! |
Needs to be rebased. I would then immediately follow up with a manufacturer supported category and capture the support email and website in the build system. |
This works for a first round, however I'd like to avoid for users to have to switch build commands several times, and as this conflicts with other labels (like rtps or fixedwing), I prefer to directly to a more permanent solution, like |
Yes the label breaks down if you want multiple variants of the same experimental board, but it seemed like an okay way to bring the issue front and center with something effectively unused/ignored for the vast majority of users (_default). I wouldn't be opposed to jumping straight to a more permanent solution, I just don't know what that is exactly. What are the top level categories, how does that work with manufacturers and official px4 standard boards? How would we handle some boards being manufacturer supported and some manufacturer's boards still being experimental? I'm not sure if actual tree placement even adds any real visibility. |
d93292d
to
ceac200
Compare
We can do:
And |
@dagar I prefer the folder structure as well. The question is if the experimental boards should have that added to their names IN ADDITION to make it really clear. |
@bkueng that part was reasonably clear, but there was talk of a level in between for manufacturer supported. If we really want the support status to be understood by the majority of developers and even users then experimental needs to be pervasive throughout the build invocations, generated artifacts, and ideally board naming internally (ver, logs, etc). I think we can make that happen easily enough with just 2 categories (and no burden on the official targets), but the line in between for manufacturer supported boards (if any) needs to be considered from the beginning.
@LorenzMeier likewise, this PR was originally the 90 second solution until we were ready to hash out all the details that now we're now getting into. Simply splitting the folders is harmless, but I don't think that alone is going to make it clear for even most developers. |
3671bad
to
3ea5970
Compare
3ea5970
to
a966679
Compare
Here's the first easy step we can take for these experimental boards and setting expectations.