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

Update/Fix naming conventions #196

Closed
commanderguy3001 opened this issue Apr 17, 2022 · 3 comments
Closed

Update/Fix naming conventions #196

commanderguy3001 opened this issue Apr 17, 2022 · 3 comments

Comments

@commanderguy3001
Copy link

commanderguy3001 commented Apr 17, 2022

Currently naming conventions don't exist/are unintuitive/aren't followed/don't make sense.

One example to name directly:
https://expresslrs.org/2.0/quick-start/transmitters/tx-diy
vs
https://expresslrs.org/2.0/quick-start/receivers/rx-diyreceivers/
should be:
https://expresslrs.org/2.0/quick-start/transmitters/diy
and
https://expresslrs.org/2.0/quick-start/receivers/diy

Notice the differences:

  • the names for receivers and transmitters are the same, as it makes sense (diy for the DIY receivers, as well as the DIY transmitters)
  • the tx- and rx- part got removed, as it doesn't make sense to have it, as it is redundant information (we already have the transmitters/ and receivers/ parts)

I think the best option for naming would be to use the actual short namings (happymodel-ep1, happymodel-ep2, namimno-flash, happymodel-es900rx, etc etc).

A couple other things that bother me:

  • the Flashing Receivers and Flashing TX Modules aren't coherent in their naming, should either be Flashing Receivers and Flashing Transmitters OR Flashing RX Modules and Flashing TX modules (I reckon the former one is easier to understand)
  • All the documented Modules have a Flashing <module> before their name which makes it quite a bit harder to read, remove that as we know that we are flashing stuff and it's redundant information
  • The documented Modules in the List on the website aren't sorted, which makes it really hard to find the exact one you wanna flash right now, as you gotta look at every single line, when it would be sorted from A-Z this would be easier
  • SPI based receivers aren't mentioned as a receiver model, might be nice to have a SPI based AIO flight computers section there too, and reference the /hardware/SPI-receivers section.
@deadbytefpv
Copy link
Collaborator

#197 has been merged.
Could you reevaluate the site and post anything that bothers you still?

@commanderguy3001
Copy link
Author

commanderguy3001 commented Apr 28, 2022

yep, looks a lot better now, only things that still bother me:

  • the lists aren't sorted A-Z (well, they are sorted until Vantac, but after that there's chaos)
  • instead of having Siyi TXes I think Siyi FM30 might be a better choice
  • for happymodel EP and frsky R9 there's a small s after the item in the list, but not for Matek R24, I think putting a * behind all of these might work the best, but that's something I don't know for sure
    • also for namimno flash RXs there are multiple versions but no s behind them
  • shouldn't it be Frsky Vantac (I thought vantac is by frsky?)
  • similar for ghost, I think adding ImmersionRC would be better suited
  • all items in the list are named by <manufacturer> <lineup> now basically, except the internal modules, those are the other way around (internal Radiomaster vs Radiomaster internal or even Radiomaster *, same for jumper)
  • some devices (ghost is one notable) don't really have a proper documentation, but that's probably a good idea to put into a new issue

@maybenikhil
Copy link
Member

Looks better now from #213 . I think we said Vantac b/c frsky doesn't have their name on it. Some still may be changed but generally formatting is better

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

No branches or pull requests

3 participants